[GitHub] [accumulo] Manno15 commented on a change in pull request #1980: Add created column to Fate Print command

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

Manno15 commented on a change in pull request #1980:
URL: https://github.com/apache/accumulo/pull/1980#discussion_r604111169



##########
File path: core/src/main/java/org/apache/accumulo/fate/ZooStore.java
##########
@@ -478,6 +480,18 @@ public Serializable getProperty(long tid, String prop) {
     }
   }
 
+  @Override
+  public String getTimestamp(long tid) {
+    verifyReserved(tid);
+
+    try {
+      Stat stat = zk.getZooKeeper().exists(getTXPath(tid), false);
+      return new Date(stat.getCtime()).toString();
+    } catch (Exception e) {
+      throw new RuntimeException(e);

Review comment:
       I can see the value in this. I was trying to match the other functions in this class with the exception handling but I can look into this more. 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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