[GitHub] metron issue #772: METRON-1209: Make stellar repl take logging properties, l...

2017-10-02 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/772 It does not and should not. If that's what you're seeing, @nickwallen then it's a bug and I should fix it. ---

[GitHub] metron issue #772: METRON-1209: Make stellar repl take logging properties, l...

2017-10-02 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/772 Does this change the default log level of the REPL when run in a deployed Metron cluster? ---

[GitHub] metron issue #772: METRON-1209: Make stellar repl take logging properties, l...

2017-10-02 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/772 +1 ---

[GitHub] metron issue #772: METRON-1209: Make stellar repl take logging properties, l...

2017-10-02 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/772 As a note : ```bash -> % mvn exec:java \ -Dexec.mainClass="org.apache.metron.stellar.common.shell.StellarShell" -Dexec.args="-l src/test/resources/log4j.properties"

[GitHub] metron issue #772: METRON-1209: Make stellar repl take logging properties, l...

2017-10-02 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/772 Oh, sorry, forgot to respond here @ottobackwards You can test it by passing in a log4j properties (let's say something that turns on DEBUG logging) file via -l and ensure that the REPL shows debug

[GitHub] metron issue #772: METRON-1209: Make stellar repl take logging properties, l...

2017-09-30 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/772 This looks good, how should it be tested? ---