William Lo created GOBBLIN-2128:
-----------------------------------
Summary: Set permission in Manifest distcp does not set all paths
correctly
Key: GOBBLIN-2128
URL: https://issues.apache.org/jira/browse/GOBBLIN-2128
Project: Apache Gobblin
Issue Type: Bug
Reporter: William Lo
There's a bug in the treemap for `SetPermissionCommitStep` where paths of the
equal file depth can be misassigned permissions due to an error in the Treemap
initialization:
{code:java}
Long.compare(o1.chars().filter(ch -> ch == '/').count(), o2.chars().filter(ch
-> ch == '/').count()) {code}
Treats any key with the same depth as equal, this could lead to incorrect
behavior when performing map operations in this map.
We want to rearrange this so that the comparator also compares the source
string.
Additionally, build off the map from the same file permissions as the ancestor
tree in copyable file so that it can stay consistent permission reading and
reduce FS calls.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
