accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] milleruntime commented on a change in pull request #326: ACCUMULO-4745 Fixed broken links in tables table on monitor
Date Thu, 01 Jan 1970 00:00:00 GMT
milleruntime commented on a change in pull request #326: ACCUMULO-4745 Fixed broken links in
tables table on monitor
URL: https://github.com/apache/accumulo/pull/326#discussion_r153568544
 
 

 ##########
 File path: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tables/TableInformation.java
 ##########
 @@ -30,7 +29,7 @@
 
   // Variable names become JSON keys
   public String tablename;
-  public Table.ID tableId;
+  public String tableId;
 
 Review comment:
   Could change the variable name, like you did below.  You would probably have to change
it on the front end as well though.  You could use tableIdString if the other is too long.
 It is just nice to know you are working with a string and not the Table.ID object.

----------------------------------------------------------------
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

Mime
View raw message