mrva/vscode-codeql
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Merge pull request #351 from jcreedcmu/jcreed/no-paginate

Browse Source 
Don't paginate at all in experimental bqrs parsing codepath
This commit is contained in:
jcreedcmu
2020-05-07 09:46:31 -04:00
committed by GitHub
parent 8b5bdbb6ef a0e60fb154
commit 53e1794b50
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
extensions/ql-vscode/src/interface.ts
View File

@@ -367,12 +367,9 @@ export class InterfaceManager extends DisposableObject {
if (EXPERIMENTAL_BQRS_SETTING.getValue()) {
resultSets = [];
// Setting pageSize very large for now for the sake of
// performance testing of vscode on containers.
const pageSize = 1e20;
const schemas = await this.cliServer.bqrsInfo(results.query.resultsPaths.resultsPath, pageSize);
const schemas = await this.cliServer.bqrsInfo(results.query.resultsPaths.resultsPath);
for (const schema of schemas["result-sets"]) {
const chunk = await this.cliServer.bqrsDecode(results.query.resultsPaths.resultsPath, schema.name, pageSize, 0)
const chunk = await this.cliServer.bqrsDecode(results.query.resultsPaths.resultsPath, schema.name);
const adaptedSchema = adaptSchema(schema);
const resultSet = adaptBqrs(adaptedSchema, chunk);
resultSets.push(resultSet);