[GitHub] nifi issue #1492: NIFI-3405 - Add uptime to JVM section in System Diagnostic...

[ijokarumawak]

Github user ijokarumawak commented on the issue:

    https://github.com/apache/nifi/pull/1492
  
    @pvillard31 Thanks for the improvement! I've tested it with a standalone 
and cluster mode.
    Although uptime was displayed as expected, I felt the NiFi Cluster dialog 
became a little bit confusing because there are two 'Uptime' columns now. One 
is in 'NODES' tab, and another is 'JVM' tab.
    
    The one in 'NODES' tab is actually a timestamp denoting when the node has 
started.
    
https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/cluster/nf-cluster-table.js#L498
    
    I'd suggest renaming the existing one in 'NODES' tab to something like 
`Started At`. How do you think?
    
    
![image](https://cloud.githubusercontent.com/assets/1107620/22921137/c48a1de6-f2db-11e6-85c9-f1e65ae565bc.png)
    
    
![image](https://cloud.githubusercontent.com/assets/1107620/22921160/dae4d68a-f2db-11e6-93bb-2bc1282ba333.png)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---