kevinrr888 commented on code in PR #5990:
URL: https://github.com/apache/accumulo/pull/5990#discussion_r2578934036
##########
core/src/main/java/org/apache/accumulo/core/client/admin/NewTableConfiguration.java:
##########
@@ -312,6 +347,7 @@ public NewTableConfiguration attachIterator(IteratorSetting
setting,
// verify that the iteratorProps assigned and the properties do not
share any keys.
checkDisjoint(properties, iteratorProps, "iterator");
}
+ iteratorSettings.put(setting, scopes);
Review Comment:
I think you're right, I didn't consider that use case.
Either way, `iteratorSettings` was removed in
db8d7f5221cddd5bf04952f4e578994e90b9f3ec so isn't a problem anymore
--
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]
