flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zentol <...@git.apache.org>
Subject [GitHub] flink issue #5357: [hotfix][JobGraph] Eliminate the conditions of parallelis...
Date Wed, 07 Feb 2018 10:49:42 GMT
Github user zentol commented on the issue:

    https://github.com/apache/flink/pull/5357
  
    @maqingxiang You are correct that the parallelism check is redundant, but that doesn't
automatically mean that it should be removed.
    
    This change imo hurts readability as the basic parallelism condition is no longer explicit,
but implicitly covered by the partitioner used.


---

Mime
View raw message