From 66712b73b0beba13a2462b39cf287dfbb711653a Mon Sep 17 00:00:00 2001 From: Ayush Saxena Date: Fri, 23 Jun 2023 19:04:09 +0530 Subject: [PATCH] TEZ-4413: permission may not be set after crash. Change-Id: I75eebcdddee21c67a5b1459f55ae22bbd4daeb83 --- .../tez/dag/history/logging/proto/DatePartitionedLogger.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tez-plugins/tez-protobuf-history-plugin/src/main/java/org/apache/tez/dag/history/logging/proto/DatePartitionedLogger.java b/tez-plugins/tez-protobuf-history-plugin/src/main/java/org/apache/tez/dag/history/logging/proto/DatePartitionedLogger.java index a569567d1e..99bc50d883 100644 --- a/tez-plugins/tez-protobuf-history-plugin/src/main/java/org/apache/tez/dag/history/logging/proto/DatePartitionedLogger.java +++ b/tez-plugins/tez-protobuf-history-plugin/src/main/java/org/apache/tez/dag/history/logging/proto/DatePartitionedLogger.java @@ -73,8 +73,7 @@ private void createDirIfNotExists(Path path) throws IOException { FileSystem fileSystem = path.getFileSystem(conf); try { if (!fileSystem.exists(path)) { - fileSystem.mkdirs(path); - fileSystem.setPermission(path, DIR_PERMISSION); + fileSystem.mkdirs(path, DIR_PERMISSION); } } catch (IOException e) { // Ignore this exception, if there is a problem it'll fail when trying to read or write.