siddharthteotia commented on code in PR #10171:
URL: https://github.com/apache/pinot/pull/10171#discussion_r1092492226
##########
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterServerQueryKillingTest.java:
##########
@@ -56,10 +56,10 @@
/**
- * Integration test that converts Avro data for 12 segments and runs queries
against it.
+ * Integration test for heap size based server query killing, this works only
for xmx4G
*/
-public class OfflineClusterQueryKillingTest extends
BaseClusterIntegrationTestSet {
- private static final Logger LOGGER =
LoggerFactory.getLogger(OfflineClusterQueryKillingTest.class);
+public class OfflineClusterServerQueryKillingTest extends
BaseClusterIntegrationTestSet {
Review Comment:
May want to rename to `OfflineServerMemBasedQueryKilingTest`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
