[GitHub] [pulsar] nicoloboschi commented on a diff in pull request #16473: [fix][test] Catch exception when update data in mockZookeeper

[GitBox <gi...@apache.org>]

nicoloboschi commented on code in PR #16473:
URL: https://github.com/apache/pulsar/pull/16473#discussion_r917964724


##########
pom.xml:
##########
@@ -100,7 +100,7 @@ flexible messaging model and an intuitive client API.</description>
       --add-opens java.management/sun.management=ALL-UNNAMED <!--JvmDefaultGCMetricsLogger-->
     </test.additional.args>
     <testReuseFork>true</testReuseFork>
-    <testForkCount>4</testForkCount>
+    <testForkCount>0</testForkCount>

Review Comment:
   this is not related to the pull. please revert



##########
build/run_unit_group.sh:
##########
@@ -64,7 +64,7 @@ alias echo='{ [[ $- =~ .*x.* ]] && trace_enabled=1 || trace_enabled=0; set +x; }
 
 # Test Groups  -- start --
 function test_group_broker_group_1() {
-  mvn_test -pl pulsar-broker -Dgroups='broker' -DtestReuseFork=true -DskipAfterFailureCount=1
+  mvn_test -pl pulsar-broker -Dgroups='broker' -DtestReuseFork=true -DskipAfterFailureCount=1 -DforkCount=0

Review Comment:
   this is not related to the pull. please revert



-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org