metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From merrimanr <...@git.apache.org>
Subject [GitHub] metron issue #737: METRON-1161: Add ability to edit parser command line opti...
Date Thu, 21 Sep 2017 14:23:29 GMT
Github user merrimanr commented on the issue:

    https://github.com/apache/metron/pull/737
  
    Changing numWorkers and numAckers default values from null to 1 had the unintended consequences
of breaking the ParserTopologyCliTest.  This is because they can no longer be overriden by
setting the topology.workers and topology.acker.executors properties in stormConfig.  I believe
this is the correct behaviour because numWorkers and numAckers are top-level properties and
should take precedence.  If this is not correct then I would argue that we shouldn't even
have them.  Can you verify this @cestella?


---

Mime
View raw message