codelipenghui commented on code in PR #18307:
URL: https://github.com/apache/pulsar/pull/18307#discussion_r1023989216
##########
pulsar-broker/src/test/java/org/apache/pulsar/broker/transaction/TransactionProduceTest.java:
##########
@@ -89,35 +89,6 @@ public void produceAndCommitTest() throws Exception {
produceTest(true);
}
- @Test
- public void testDeleteNamespaceBeforeCommit() throws Exception {
Review Comment:
Why we should delete the entire test?
If the expected behavior is the transaction is not allowed to commit. We can
just change the test to check the expected exception.
--
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]
