[GitHub] metron issue #1143: METRON-1713: PCAP UI - Add a way to kill a pcap job

2018-08-09 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/1143 This is merged. Can you close @tiborm? ---

[GitHub] metron issue #1143: METRON-1713: PCAP UI - Add a way to kill a pcap job

2018-08-09 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/1143 @tiborm can you merge the latest feature branch in? @justinleet are there any other outstanding requests or are you satisfied? ---

[GitHub] metron issue #1143: METRON-1713: PCAP UI - Add a way to kill a pcap job

2018-08-08 Thread tiborm
Github user tiborm commented on the issue: https://github.com/apache/metron/pull/1143 @merrimanr Yes, it contains the latest state of METRON-1712 and feature/METRON-1554-pcap-query-panel. ---

[GitHub] metron issue #1143: METRON-1713: PCAP UI - Add a way to kill a pcap job

2018-08-08 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/1143 Sigh, my inability to get my browser in a consistent state strikes again. Clearing everything out and trying again shows the behavior you describe. ---

[GitHub] metron issue #1143: METRON-1713: PCAP UI - Add a way to kill a pcap job

2018-08-08 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/1143 @justinleet I am seeing different behavior. When I submit a query, the cancel button appears but is disabled. I am not able to kill a job in the middle of submission. After the job submission

[GitHub] metron issue #1143: METRON-1713: PCAP UI - Add a way to kill a pcap job

2018-08-08 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/1143 Does this PR also contain METRON-1712? I don't see anything in the PR description stating that and I see several changes that are not related to this PR. ---

[GitHub] metron issue #1143: METRON-1713: PCAP UI - Add a way to kill a pcap job

2018-08-08 Thread tiborm
Github user tiborm commented on the issue: https://github.com/apache/metron/pull/1143 Thanks, Justin! I updated the PR with a small fix. ---