[ 
https://issues.apache.org/jira/browse/FLINK-10631?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16671899#comment-16671899
 ] 

ASF GitHub Bot commented on FLINK-10631:
----------------------------------------

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


> Update jepsen tests to run with multiple slots
> ----------------------------------------------
>
>                 Key: FLINK-10631
>                 URL: https://issues.apache.org/jira/browse/FLINK-10631
>             Project: Flink
>          Issue Type: Task
>          Components: Tests
>    Affects Versions: 1.7.0
>            Reporter: Till Rohrmann
>            Assignee: Till Rohrmann
>            Priority: Critical
>              Labels: pull-request-available
>             Fix For: 1.7.0
>
>
> After fixing FLINK-9455, we should update the Jepsen tests to run with 
> multiple slots per {{TaskExecutor}}.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to