steveloughran commented on PR #5872: URL: https://github.com/apache/hadoop/pull/5872#issuecomment-1682558300
latest patch removed the redundant method *and* updates the auditing docs to cover the migration to the new option, plus any other updates needed. Note: storediag updated to print the new audit option and those files on the classpath to list what to automatically load too. @ahmarsuhail if people really wanted to plug in handling here, maybe we should say "if the class implements Configurable we call setConf on it". that way people can add dynamic configuration through hadoop/spark jobs. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
