milleruntime commented on a change in pull request #352: ACCUMULO-4771 Implement DataTables in Monitor URL: https://github.com/apache/accumulo/pull/352#discussion_r160251514
########## File path: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tables/TableInformationList.java ########## @@ -21,15 +21,15 @@ /** * - * Generates a list with table information + * Generates a list with table information. This is mainly used to populate DataTables Review comment: Good point. I put this comment to explain the changes in the class below but perhaps I should just comment next to the "data" field as to why its called something as generic as data. I tried to get DataTables to work with the JSON object as "table" but couldn't and it wasn't pretty. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
