codelipenghui commented on a change in pull request #14809:
URL: https://github.com/apache/pulsar/pull/14809#discussion_r833454816



##########
File path: 
pulsar-broker/src/test/java/org/apache/pulsar/broker/transaction/TransactionTest.java
##########
@@ -828,4 +831,30 @@ public void 
testNotChangeMaxReadPositionAndAddAbortTimesWhenCheckIfNoSnapshot()
         Assert.assertEquals(changeMaxReadPositionAndAddAbortTimes.get(), 0L);
 
     }
+
+    @Test
+    public void testAutoCreateSchemaForTransactionSnapshot() throws Exception {
+        String namespace = TENANT + "/ns2";
+        String topic = namespace + "/test";
+        
pulsarServiceList.get(0).getConfiguration().setAllowAutoUpdateSchemaEnabled(false);

Review comment:
       The request might point to other brokers, only disabling schema 
auto-update for 1 broker will not cover the changes well, you can try to use 
the admin API to disable the schema auto-update or disable all the brokers.




-- 
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]


Reply via email to