kfaraz commented on code in PR #18786:
URL: https://github.com/apache/druid/pull/18786#discussion_r2575542999
##########
integration-tests/src/test/java/org/apache/druid/tests/query/ITQueryRetryTestOnMissingSegments.java:
##########
@@ -52,12 +51,11 @@
* This class tests the query retry on missing segments. A segment can be
missing in a historical during a query if
* the historical drops the segment after the broker issues the query to the
historical. To mimic this case, this
* test spawns a historical modified for testing. This historical announces
all segments assigned, but doesn't serve
- * all of them always. Instead, it can report missing segments for some
- * segments. See {@link ServerManagerForQueryErrorTest} for more details.
+ * all of them always. Instead, it can report missing segments for some
segments.
* <p>
* To run this test properly, the test group must be specified as {@link
TestNGGroup#QUERY_RETRY}.
*/
-@Test(groups = TestNGGroup.QUERY_RETRY)
+@Test(groups = TestNGGroup.QUERY_RETRY, enabled = false) // TODO: Temporarily
disabled due to migration to embedded suite.
@Guice(moduleFactory = DruidTestModuleFactory.class)
public class ITQueryRetryTestOnMissingSegments
Review Comment:
Thanks!
--
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]
