Optimize Tabular View Code about aqa-test-tools HOT 2 OPEN

I feel this is not only a specific function is used or not. There are lots of problems. Below are some examples:

• Instead of writing code to filter and find the latest run, can we query on date?
  https://github.com/AdoptOpenJDK/openjdk-test-tools/blob/69e586c031d71e49d694edd944af9e591cd10ba2/TestResultSummaryService/routes/getTabularData.js#L47-L67

• why loop through all info and string concat into benchmarkNVM , then split benchmarkNVM get the values back?

benchmarkNVM = testResultsObject.aggregateInfo[aggregateIndex].benchmarkName + ',' + testResultsObject.aggregateInfo[aggregateIndex].benchmarkVariant + "," + testResultsObject.aggregateInfo[aggregateIndex].metrics[metric].name;

• lots of unused variables: https://github.com/AdoptOpenJDK/openjdk-test-tools/blob/69e586c031d71e49d694edd944af9e591cd10ba2/TestResultSummaryService/routes/getTabularDropdown.js#L8

...

Before we dive into fixing the code, I would like to take a step back and go over the design of the Tabular View and understand the use cases. To me, both client TabularView.jsx and server logic for Tabular View seem convoluted.

from aqa-test-tools.

@llxia After some initial review for Tabular View, we didn't end up using the snippet with distinct mentioned above. This issue was created a few days before the final changes were merged, so we have cleaned up and optimized most things already.

Mentioning some relevant snippets below. If you have any specific suggestions on how to optimize the queries further, please mention it. Thanks!

https://github.com/AdoptOpenJDK/openjdk-test-tools/blob/69e586c031d71e49d694edd944af9e591cd10ba2/TestResultSummaryService/routes/getTabularDropdown.js#L18-L20

https://github.com/AdoptOpenJDK/openjdk-test-tools/blob/69e586c031d71e49d694edd944af9e591cd10ba2/TestResultSummaryService/routes/getTabularData.js#L28-L29

https://github.com/AdoptOpenJDK/openjdk-test-tools/blob/69e586c031d71e49d694edd944af9e591cd10ba2/TestResultSummaryService/routes/getTabularData.js#L69

from aqa-test-tools.