bharatviswa504 commented on a change in pull request #855:
URL: https://github.com/apache/hadoop-ozone/pull/855#discussion_r416006125
##########
File path:
hadoop-hdds/common/src/main/java/org/apache/hadoop/ozone/OzoneConsts.java
##########
@@ -329,4 +329,8 @@ private OzoneConsts() {
public static final String GDPR_ALGORITHM = "algorithm";
+ // Transaction Info
+ public static final String TRANSACTION_INFO_KEY = "TRANSACTIONINFO";
+ public static final String TRANSACTION_INFO_SPLIT_KEY = "-";
+
Review comment:
Technically we don't need it. As in container DB we have iterators to
iterate the keys in container, but there are some keys like BCSID, blockCount
which are not blocks, so to avoid this they have a filter to skip these keys
during iteration (But the same can be done without # also. Might be there it is
done to know that these are system defined keys and a way to distinguish. (This
is just my understanding, and followed the same when implementing HDDS-3217).
In OM, we have tables for each. But I can use the # prefix.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
