[ https://issues.apache.org/jira/browse/GOBBLIN-2208?focusedWorklogId=970125&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-970125 ]
ASF GitHub Bot logged work on GOBBLIN-2208: ------------------------------------------- Author: ASF GitHub Bot Created on: 21/May/25 07:23 Start Date: 21/May/25 07:23 Worklog Time Spent: 10m Work Description: abhishekmjain commented on code in PR #4117: URL: https://github.com/apache/gobblin/pull/4117#discussion_r2099557166 ########## gobblin-utility/src/test/java/org/apache/gobblin/util/HadoopUtilsTest.java: ########## @@ -333,4 +344,166 @@ public void testMoveToTrash() throws IOException { Assert.assertFalse(fs.exists(hadoopUtilsTestDir)); Assert.assertTrue(fs.exists(trashPath)); } + + @Test + public void testEnsureDirectoryExistsWithAclPreservation() throws Exception { + final Path testDir = new Path(new Path(TEST_DIR_PATH), "HadoopUtilsTestDir"); + FileSystem fs = Mockito.mock(FileSystem.class); Review Comment: instead of executing all tests with Mockito, can we use below FS like in `testRenameRecursively` test to ensure the tests are modifying real directories? `FileSystem fs = FileSystem.getLocal(new Configuration());` Issue Time Tracking ------------------- Worklog Id: (was: 970125) Time Spent: 1h (was: 50m) > ACL Mismatch between source and dest for Manifest based copy > ------------------------------------------------------------ > > Key: GOBBLIN-2208 > URL: https://issues.apache.org/jira/browse/GOBBLIN-2208 > Project: Apache Gobblin > Issue Type: Bug > Reporter: Vaibhav Singhal > Priority: Major > Time Spent: 1h > Remaining Estimate: 0h > > During manifest based copy due to default ACL inheritence in the destination, > after manifest based copy source and destination end up having different > ACL's for source and destination -- This message was sent by Atlassian Jira (v8.20.10#820010)