apurtell commented on a change in pull request #371: HBASE-22648 : Introducing 
Snapshot TTL
URL: https://github.com/apache/hbase/pull/371#discussion_r304136426
 
 

 ##########
 File path: 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/SnapshotDescription.java
 ##########
 @@ -139,15 +146,27 @@ public long getCreationTime() {
     return this.creationTime;
   }
 
+  // get snapshot ttl in sec
+  public long getTtl() {
 
 Review comment:
   Getters and setters for ttl is fine even if also using generic attribute map 
with getters and setters for elements of same. 

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to