New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HIVE-24944: When the default engine of the hiveserver is MR and the t… #2204
Conversation
Hi @anishek, could you please take another look on the changes? thanks. |
Hey @thejasmn, @kgyrtkirk could you please take another look if have secs? |
Looks alright. Can you do me a quick favor and switch statement it? switch(engineInSessionConf) {
case "tez":
case "mr":
default:
} |
…ez engine is set by the client, the client TEZ progress log cannot be printed normally(ZhangQidong reviewed by Zhihua Deng)
Hi @belugabehr @dengzhhu653 @thejasmn @kgyrtkirk @anishek |
case "spark": | ||
mapper = new SparkProgressMonitorStatusMapper(); | ||
break; | ||
default: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: maybe we could remove the default
execution path here?
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
What changes were proposed in this pull request?
See the: https://issues.apache.org/jira/browse/HIVE-24944
Why are the changes needed?
When the default engine of the hiveserver is MR and the tez engine is set by the client, the client TEZ progress log cannot be printed normally
Does this PR introduce any user-facing change?
No
How was this patch tested?
Remote cluster.