HanumathRao commented on a change in pull request #1762: [DRILL-7191 /
DRILL-7026]: RM state blob persistence in Zookeeper and Integration of
Distributed queue configuration with Planner
URL: https://github.com/apache/drill/pull/1762#discussion_r283471547
##########
File path:
exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZookeeperClient.java
##########
@@ -301,6 +324,54 @@ public void put(final String path, final byte[] data,
DataChangeVersion version)
}
}
+ public void putAsTransaction(Map<String, byte[]> pathsWithData) {
+ putAsTransaction(pathsWithData, null);
+ }
+
+ /**
+ * Puts the given sets of blob and their data's in a transactional manner.
It expects all the blob path to exist
+ * before calling this api.
+ * @param pathsWithData - map of blob paths to update and the final data
+ * @param version - version holder
+ */
+ public void putAsTransaction(Map<String, byte[]> pathsWithData,
DataChangeVersion version) {
Review comment:
Do we currently use non null version. If not then can you please mention it
in the comment that this is needed for future use.
----------------------------------------------------------------
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
