Skip to content

Commit 53e1794

Browse files
authored
Merge pull request #351 from jcreedcmu/jcreed/no-paginate
Don't paginate at all in experimental bqrs parsing codepath
2 parents 8b5bdbb + a0e60fb commit 53e1794

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

extensions/ql-vscode/src/interface.ts

+2-5
Original file line numberDiff line numberDiff line change
@@ -367,12 +367,9 @@ export class InterfaceManager extends DisposableObject {
367367

368368
if (EXPERIMENTAL_BQRS_SETTING.getValue()) {
369369
resultSets = [];
370-
// Setting pageSize very large for now for the sake of
371-
// performance testing of vscode on containers.
372-
const pageSize = 1e20;
373-
const schemas = await this.cliServer.bqrsInfo(results.query.resultsPaths.resultsPath, pageSize);
370+
const schemas = await this.cliServer.bqrsInfo(results.query.resultsPaths.resultsPath);
374371
for (const schema of schemas["result-sets"]) {
375-
const chunk = await this.cliServer.bqrsDecode(results.query.resultsPaths.resultsPath, schema.name, pageSize, 0)
372+
const chunk = await this.cliServer.bqrsDecode(results.query.resultsPaths.resultsPath, schema.name);
376373
const adaptedSchema = adaptSchema(schema);
377374
const resultSet = adaptBqrs(adaptedSchema, chunk);
378375
resultSets.push(resultSet);

0 commit comments

Comments
 (0)