YARN-6169 message on empty configuration file improved (#4952)

riyakhdl · web-flow · commit 07581f1ab2cf · 2022-10-03T23:31:06.000-04:00
diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/native/container-executor/impl/configuration.c b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/native/container-executor/impl/configuration.c
@@ -546,8 +546,8 @@ int read_config(const char *file_path, struct configuration *cfg) {
 
   if (cfg->size == 0) {
     free_configuration(cfg);
-    fprintf(ERRORFILE, "Invalid configuration provided in %s\n", file_path);
-    return INVALID_CONFIG_FILE;
+    fprintf(ERRORFILE, "Empty configuration file provided %s\n", file_path);
+    exit(INVALID_CONFIG_FILE);
   }
   return 0;
 }

Original file line number	Diff line number	Diff line change
`@@ -546,8 +546,8 @@ int read_config(const char file_path, struct configuration cfg) {`
`546`	`546`
`547`	`547`	`if (cfg->size == 0) {`
`548`	`548`	`free_configuration(cfg);`
`549`		`- fprintf(ERRORFILE, "Invalid configuration provided in %s\n", file_path);`
`550`		`- return INVALID_CONFIG_FILE;`
	`549`	`+ fprintf(ERRORFILE, "Empty configuration file provided %s\n", file_path);`
	`550`	`+ exit(INVALID_CONFIG_FILE);`
`551`	`551`	`}`
`552`	`552`	`return 0;`
`553`	`553`	`}`