metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From merrimanr <>
Subject [GitHub] metron issue #737: METRON-1161: Add ability to edit parser command line opti...
Date Wed, 20 Sep 2017 22:04:57 GMT
Github user merrimanr commented on the issue:
    Just pushed out a commit that addresses @ottobackwards comments and fixes the NUM_WORKERS
and NUM_ACKERS issue.  I also added a warning that appears when a storm setting changes, telling
the user that the topology needs to be restarted.  It may seem redundant that the same message
appears for each field but we might want a different message in the future once we expose
a way to rebalance using zookeeper values.  At that point this pattern would make sense and
I don't think it necessarily looks too bad now.
    @nickwallen I also started a discussion on the other points you brought up.  Hopefully
whatever conclusion we come to there can be a follow-on JIRA.


View raw message