[GitHub] milleruntime opened a new pull request #352: ACCUMULO-4771 Implement DataTables in Monitor

[GitBox <gi...@apache.org>]

milleruntime opened a new pull request #352: ACCUMULO-4771 Implement DataTables in Monitor
URL: https://github.com/apache/accumulo/pull/352
 
 
   * Drastically reduces our custom JS code.  
   * Created a new rest endpoint (/dataTables) that could replace a lot of the code in TablesResource
   * Made improvements to the TableInformation JSON object
   * Re-purposed TableInformationList for use in DataTables (was only used in XmlInformation)
   * Imported datatables plugins: ellipsis (ellipsis.js) and scientific notation sort (in global.js) to make the really small numbers pretty and sortable
   * select2 is eliminated but currently no way to search on default (empty) namespace.  If we still want this I believe this can be done using type based search https://datatables.net/plug-ins/filtering/ but I haven't played around with it yet.
   
   I only implemented DataTables for the "tables" page so I'm soliciting feedback before making changes to all tables on the Monitor

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services