codope commented on pull request #3630: URL: https://github.com/apache/hudi/pull/3630#issuecomment-949411126
> @codope : Did you get a chance to repro this as vinoth suggested? It does not reproduce in local docker setup. It requires Tez. So, I setup on EMR and I could reproduce the issue. The patch fixes it by setting the right value of conf. Upon enabling info logging, I realized that it does not even get inside the if-block in case of Tez. This patch is good to be merged. -- 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: commits-unsubscr...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
