You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/08/17 15:57:00 UTC

[GitHub] [accumulo] milleruntime opened a new pull request #2234: Create AccumuloDataVersion

milleruntime opened a new pull request #2234:
URL: https://github.com/apache/accumulo/pull/2234


   * Move data version from ServerConstants to AccumuloDataVersion and added
   javadoc description
   * Create AccumuloDataVersion.get() for getting data version
   * Move other constants from ServerConstants to Constants
   * Rename ServerConstants to ServerDirs


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

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [accumulo] EdColeman commented on a change in pull request #2234: Create AccumuloDataVersion

Posted by GitBox <gi...@apache.org>.
EdColeman commented on a change in pull request #2234:
URL: https://github.com/apache/accumulo/pull/2234#discussion_r690531723



##########
File path: core/src/main/java/org/apache/accumulo/core/Constants.java
##########
@@ -23,6 +23,11 @@
 public class Constants {
 

Review comment:
       ```suggestion
      // defines Accumulo data version constants
   ```




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

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [accumulo] milleruntime merged pull request #2234: Create AccumuloDataVersion

Posted by GitBox <gi...@apache.org>.
milleruntime merged pull request #2234:
URL: https://github.com/apache/accumulo/pull/2234


   


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

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [accumulo] EdColeman commented on a change in pull request #2234: Create AccumuloDataVersion

Posted by GitBox <gi...@apache.org>.
EdColeman commented on a change in pull request #2234:
URL: https://github.com/apache/accumulo/pull/2234#discussion_r690531723



##########
File path: core/src/main/java/org/apache/accumulo/core/Constants.java
##########
@@ -23,6 +23,11 @@
 public class Constants {
 

Review comment:
       ```suggestion
      // defines Accumulo data version constants
   ```




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

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [accumulo] milleruntime merged pull request #2234: Create AccumuloDataVersion

Posted by GitBox <gi...@apache.org>.
milleruntime merged pull request #2234:
URL: https://github.com/apache/accumulo/pull/2234


   


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

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org