Skip to content

Commit

Permalink
Merge pull request #112 from perftool-incubator/mega-q-limits
Browse files Browse the repository at this point in the history
Limit batched queries to 10 at a time
  • Loading branch information
atheurer authored Aug 2, 2024
2 parents 8c0391e + 4bd4478 commit 6488be5
Showing 1 changed file with 18 additions and 4 deletions.
22 changes: 18 additions & 4 deletions queries/cdmq/get-result-summary.js
Original file line number Diff line number Diff line change
Expand Up @@ -149,8 +149,11 @@ runIds.forEach((runId) => {
}

// build the sets for the mega-query
var metricDataSetsChunks = [];
var batchedQuerySize = 10;
var benchmarks = benchName.split(',');
var sets = [];
var chunkNum = 0;
for (var i = 0; i < benchIterations.length; i++) {
for (var j = 0; j < iterSampleIds[i].length; j++) {
var primaryMetrics = list(iterPrimaryMetrics[i]);
Expand Down Expand Up @@ -181,12 +184,21 @@ runIds.forEach((runId) => {
breakout: []
};
sets.push(set);
if (sets.length == batchedQuerySize) {
// Submit a chunk of the query and save the result
metricDataSetsChunks[chunkNum] = cdm.getMetricDataSets(program.url, sets);
chunkNum++;
sets = [];
}
}
}
}

// do the mega-query
var metricDataSets = cdm.getMetricDataSets(program.url, sets);
if (sets.length > 0) {
// Submit a chunk of the query and save the result
metricDataSetsChunks[chunkNum] = cdm.getMetricDataSets(program.url, sets);
chunkNum++;
sets = [];
}

// output the results
var data = {};
Expand Down Expand Up @@ -282,7 +294,9 @@ runIds.forEach((runId) => {
var primaryMetrics = list(iterPrimaryMetrics[i]);
for (var k = 0; k < primaryMetrics.length; k++) {
var sourceType = primaryMetrics[k].split('::');
msampleVal = parseFloat(metricDataSets[idx].values[''][0].value);
var thisChunk = Math.floor(idx / batchedQuerySize);
var thisIdx = idx % batchedQuerySize;
msampleVal = parseFloat(metricDataSetsChunks[thisChunk][thisIdx].values[''][0].value);
if (allBenchMsampleVals[k] == null) {
allBenchMsampleVals[k] = [];
}
Expand Down

0 comments on commit 6488be5

Please sign in to comment.