kevinrr888 commented on code in PR #6040:
URL: https://github.com/apache/accumulo/pull/6040#discussion_r2721583963
##########
server/manager/src/main/java/org/apache/accumulo/manager/FateServiceHandler.java:
##########
@@ -322,22 +323,27 @@ public void executeFateOperation(TInfo tinfo,
TCredentials c, long opid, FateOpe
Map<String,String> propertiesToSet = new HashMap<>();
Set<String> propertiesToExclude = new HashSet<>();
+ // dest table will have the dest namespace props + src table props:
need to check provided
+ // options to set for conflicts with this
+ var srcTableConfigIterProps =
Review Comment:
Good catch. Yeah, I wasn't sure how to get the table props only here (not
merged view).
I looked over your suggestion and it looks good. I added it in
f72d9a4834d4343efccc68658780f2fde5f14b02
--
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]
