abhishekrb19 commented on code in PR #16208: URL: https://github.com/apache/druid/pull/16208#discussion_r1542189015
########## server/src/test/java/org/apache/druid/metadata/IndexerSQLMetadataStorageCoordinatorTest.java: ########## @@ -935,7 +935,12 @@ public void testTransactionalAnnounceFailDbNullWantNotNull() throws IOException new ObjectMetadata(ImmutableMap.of("foo", "bar")), new ObjectMetadata(ImmutableMap.of("foo", "baz")) ); - Assert.assertEquals(SegmentPublishResult.fail("java.lang.RuntimeException: Failed to update the metadata Store. The new start metadata is ahead of last commited end state."), result1); + Assert.assertEquals( + SegmentPublishResult.fail( + "java.lang.RuntimeException: Failed to update the metadata Store." Review Comment: Ack, I will update the callers to throw DruidException and update the tests accordingly. -- 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: commits-unsubscr...@druid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org