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-27055: hive-exec typos part 3 #4035
Conversation
# Conflicts: # ql/src/java/org/apache/hadoop/hive/ql/exec/repl/ReplDumpTask.java
@abstractdog any chance you could review it? |
@@ -798,7 +798,7 @@ && shouldBootstrapDumpAcidTable(table.getTableName())) { | |||
return !ReplUtils.tableIncludedInReplScope(work.oldReplScope, table.getTableName()); | |||
} | |||
|
|||
private boolean isTableSatifiesConfig(Table table) { | |||
private boolean isTableSatisfiesConfig(Table table) { |
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.
if we're about grammar, and this is a private method (we won't break anything), isn't it "doesTableSatisfyConfig instead?
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.
LGTM, 1 minor change request
Kudos, SonarCloud Quality Gate passed! |
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
…ed by Laszlo Bodor)
…ed by Laszlo Bodor)
What changes were proposed in this pull request?
fix typos in comments and code
Why are the changes needed?
to improve code quality
Does this PR introduce any user-facing change?
No (except debug logging)
How was this patch tested?
build