snleee commented on code in PR #9890:
URL: https://github.com/apache/pinot/pull/9890#discussion_r1041373092
##########
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/MergeRollupMinionOfflineClusterIntegrationTest.java:
##########
@@ -65,9 +65,9 @@
/**
- * Integration test for minion task of type "MergeRollupTask"
+ * Integration test for minion task of type "MergeRollupTask" configured on
offline tables
*/
-public class MergeRollupMinionClusterIntegrationTest extends
BaseClusterIntegrationTest {
+public class MergeRollupMinionOfflineClusterIntegrationTest extends
BaseClusterIntegrationTest {
Review Comment:
By the way, is it possible to merge offline/realtime tests using 1
integration test since we are based on `BaseClusterIntegrationTest` for both
tests? Integration test is expensive to set up the env.
--
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]
