steveloughran commented on pull request #2732: URL: https://github.com/apache/hadoop/pull/2732#issuecomment-804420292
LGTM. I'm going to make one final change, but +1 this PR pending that change anyway. Can you put the CommonCallableSupplier into `org.apache.hadoop.util.functional` . Sorry, should have thought of this earlier. Thats where I'm trying to unify the API for functional APIs in hadoop with IOE support -and this is clearly part of it. Nothing else, just a move of the class. +1 pending that change. -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
