[GitHub] milleruntime commented on a change in pull request #352: ACCUMULO-4771 Implement DataTables in Monitor

2018-01-09 Thread GitBox
milleruntime commented on a change in pull request #352: ACCUMULO-4771 Implement DataTables in Monitor URL: https://github.com/apache/accumulo/pull/352#discussion_r160547497 ## File path: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tables/TableInformationList.java

[jira] [Assigned] (ACCUMULO-4158) Investigate using Google's error-prone

2018-01-09 Thread Mark Owens (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4158?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Owens reassigned ACCUMULO-4158: Assignee: Mark Owens > Investigate using Google's error-prone >

[GitHub] milleruntime commented on issue #352: ACCUMULO-4771 Implement DataTables in Monitor

2018-01-09 Thread GitBox
milleruntime commented on issue #352: ACCUMULO-4771 Implement DataTables in Monitor URL: https://github.com/apache/accumulo/pull/352#issuecomment-356385931 I am going to close this PR so I can rebase, dropping the unnecessary commits. --

[GitHub] milleruntime commented on issue #352: ACCUMULO-4771 Implement DataTables in Monitor

2018-01-09 Thread GitBox
milleruntime commented on issue #352: ACCUMULO-4771 Implement DataTables in Monitor URL: https://github.com/apache/accumulo/pull/352#issuecomment-356385440 The more I think about wrapping the current REST endpoints, the more I realize that just complicates the backend. I am thinking the Mo

[GitHub] milleruntime commented on issue #352: ACCUMULO-4771 Implement DataTables in Monitor

2018-01-09 Thread GitBox
milleruntime commented on issue #352: ACCUMULO-4771 Implement DataTables in Monitor URL: https://github.com/apache/accumulo/pull/352#issuecomment-356381781 What do you guys think of 5373bb25fb49a83fd8fa1f565b683fe62738110f ? It is a wrapper for DataTables so any JSON object we want to ren