https://pinot.apache.org/ logo
#pinot-dev
Title
# pinot-dev
j

Jack

04/15/2021, 8:36 PM
Yes, we have included all the types of queries at that time and added more coverage on that
m

Mayank

04/15/2021, 8:37 PM
I meant do we have a code cov report that shows the increase in coverage?
j

Jack

04/15/2021, 8:39 PM
There is a coverage report in that PR, which shows the coverage increased by 6.74% https://github.com/apache/incubator-pinot/pull/6181
m

Mayank

04/15/2021, 8:42 PM
Ok. Let's see what the overall test runtime is after we fix the network interface issue. Right now it seems to be taking several hours and still stuck.
j

Jack

04/15/2021, 11:54 PM
@User @User I just opened a PR to further reduce the number of queries from 500 to 200: https://github.com/apache/incubator-pinot/pull/6802
m

Mayank

04/15/2021, 11:58 PM
Thanks @User, what's the new runtime, and do we still keep the previous increase in coverage?
j

Jack

04/16/2021, 12:00 AM
I removed some of the queries in each query pattern. The code coverage report hasn’t been generated yet. Hopefully it will remain the same.
m

Mayank

04/16/2021, 12:08 AM
👍