tillrohrmann commented on a change in pull request #6978: [FLINK-10631] Set 
number of slots per TM to 3 for Jepsen tests
URL: https://github.com/apache/flink/pull/6978#discussion_r230124525
 
 

 ##########
 File path: flink-jepsen/src/jepsen/flink/db.clj
 ##########
 @@ -293,7 +293,6 @@
        "-Djobmanager.rpc.port=6123 "
        "-Dmesos.resourcemanager.tasks.mem=2048 "
        "-Dtaskmanager.heap.mb=2048 "
-       "-Dtaskmanager.numberOfTaskSlots=2 "
 
 Review comment:
   I think Mesos will pick up the configuration settings from the Flink 
configuration. Therefore, there should be no need to explicitly override it 
here. The same may apply to the other configuration settings as well. But I 
would like to address this in a separate issue.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to