[GitHub] nifi issue #3021: NIFI-5600 - Restore node location display on queue listing

2018-10-09 Thread pvillard31 
Github user pvillard31 commented on the issue:

https://github.com/apache/nifi/pull/3021
  
Thanks much @mcgilman !


---

[GitHub] nifi issue #3021: NIFI-5600 - Restore node location display on queue listing

2018-10-09 Thread mcgilman 
Github user mcgilman commented on the issue:

https://github.com/apache/nifi/pull/3021
  
Provided an alternate solution here: 
https://github.com/apache/nifi/pull/3055


---

[GitHub] nifi issue #3021: NIFI-5600 - Restore node location display on queue listing

2018-10-09 Thread mcgilman 
Github user mcgilman commented on the issue:

https://github.com/apache/nifi/pull/3021
  
@pvillard31 Thanks for the PR! I can see what is happening here and I think 
a slightly different solution is required. With the introduction of NIFI-5208 
we should be watching the cluster status actively so we can track the 
disconnected state. Because this state can change, I think we need to 
reconsider the columns we show when the listing is opened (not when the UI is 
bootstrapped). We actually have the same issue showing component state. I am 
going to open up another PR with this alternate solution. 


---

[GitHub] nifi issue #3021: NIFI-5600 - Restore node location display on queue listing

2018-10-09 Thread mcgilman 
Github user mcgilman commented on the issue:

https://github.com/apache/nifi/pull/3021
  
Sorry for missing the mention... Will review


---

[GitHub] nifi issue #3021: NIFI-5600 - Restore node location display on queue listing

2018-09-23 Thread pvillard31 
Github user pvillard31 commented on the issue:

https://github.com/apache/nifi/pull/3021
  
@mcgilman @scottyaslan - friendly ping to be sure I didn't do something 
against the purpose of NIFI-5208 (#2740)


---