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-27752: Remove DagUtils duplicate class #4756
Conversation
@@ -227,6 +228,14 @@ public Object process(Node nd, Stack<Node> stack, NodeProcessorCtx procCtx, | |||
return null; | |||
} | |||
} | |||
public static String getQueryName(Configuration conf) { |
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.
you haven't needed to change any reference? isn't this used at all? if so, I'm fine with removing it :)
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.
Yup, no usage that the ide could detect as well, will remove it :)
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information The version of Java (11.0.8) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. |
f52b59c
to
5da6ffd
Compare
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.
+1
previous precommit passed, it makes no sense to wait for this one |
…hur reviewed by Laszlo Bodor)
What changes were proposed in this pull request?
Remove DagUtils duplicate class, after DagUtils.java in tez package
Why are the changes needed?
Code cleanup
Does this PR introduce any user-facing change?
No
Is the change a dependency upgrade?
No
How was this patch tested?
Existing Unit test