dlmarion commented on code in PR #5334:
URL: https://github.com/apache/accumulo/pull/5334#discussion_r1956516968
##########
server/manager/src/main/java/org/apache/accumulo/manager/tableOps/clone/CloneZookeeper.java:
##########
@@ -37,20 +35,12 @@ class CloneZookeeper extends ManagerRepo {
public CloneZookeeper(CloneInfo cloneInfo, ClientContext context)
throws NamespaceNotFoundException {
this.cloneInfo = cloneInfo;
- this.cloneInfo.namespaceId = Namespaces.getNamespaceId(context,
Review Comment:
Oh, are you saying that this may cause pre-2.1.4 Clone fate operations to
fail, and that I should still check and set this if it's null in CloneInfo?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
