accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] ctubbsii commented on a change in pull request #376: ACCUMULO-4798 optimized stat in ZooCache
Date Thu, 01 Jan 1970 00:00:00 GMT
ctubbsii commented on a change in pull request #376: ACCUMULO-4798 optimized stat in ZooCache
URL: https://github.com/apache/accumulo/pull/376#discussion_r167109206
 
 

 ##########
 File path: fate/src/test/java/org/apache/accumulo/fate/zookeeper/ZooCacheTest.java
 ##########
 @@ -80,9 +80,9 @@ private void testGet(boolean fillStat) throws Exception {
     if (fillStat) {
       myStat = new ZcStat();
     }
-    long now = System.currentTimeMillis();
+    final long ephemeralOwner = 123456789l;
 
 Review comment:
   Using Random here might make this more clear that this number doesn't matter... we're just
using it to ensure the number we put in is the same one we see later. But, a brief comment
to that would also work.

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