Skip to content

Fix miscellaneous typos in comments, docs, and a test script - #6742

Open
Akanksha-kedia wants to merge 1 commit into
apache:masterfrom
Akanksha-kedia:fix/misc-typos-comments-docs
Open

Fix miscellaneous typos in comments, docs, and a test script#6742
Akanksha-kedia wants to merge 1 commit into
apache:masterfrom
Akanksha-kedia:fix/misc-typos-comments-docs

Conversation

@Akanksha-kedia

Copy link
Copy Markdown

What changes were proposed in this pull request?

Fixes several unrelated small misspellings found while reviewing comments and docs:

  • metastore-benchmarks README: <optins> -> <options> (2 occurrences)
  • .github/pull_request_template.md: "reproducable" -> "reproducible"
  • Hive.java: "Partiton" -> "Partition", "transfering" -> "transferring", "differnent" -> "different"
  • PartitionIterable.java: "seperate" -> "separate"
  • PartitionTree.java: "bofore" -> "before"
  • StatsPublisher.java: "statics" -> "statistics" (Javadoc)
  • VariantBuilder.java: "calcualtion" -> "calculation"
  • HiveMaterializedViewsRegistry.java: "partion" -> "partition"
  • HiveUtils.java: "characeters" -> "characters"
  • BasicStatsNoJobTask.java: "intresting" -> "interesting" (exception message)
  • debug/Utils.java: "directoty" -> "directory"
  • HiveClientCache.java: "wont" -> "won't"
  • metastore-validation-test.sh: "occurances" -> "occurrences", "retrive" -> "retrieve" (x4), "begining" -> "beginning", "detemine" -> "determine", "thrid" -> "third" (x2) — comment text only, the num_occurances() function name is left unchanged to avoid touching code structure

No JIRA ticket filed since this is a comment/doc-only, non-functional change with no behavior impact — happy to file one if a reviewer prefers that.

Why are the changes needed?

Spelling correctness in comments, docs, and a dev/test script. No behavior change.

Does this PR introduce any user-facing change?

No. Only comment/doc/string text changes. The one exception message text change (BasicStatsNoJobTask.java) is not asserted by any existing test.

How was this patch tested?

Comment/doc/string-only changes; no test behavior is affected.

Fixes several unrelated misspellings found in Javadoc comments, an
exception message, the PR template, and a metastore validation test
script:

- metastore-benchmarks README: <optins> -> <options> (2 occurrences)
- .github/pull_request_template.md: reproducable -> reproducible
- Hive.java: Partiton -> Partition, transfering -> transferring,
  differnent -> different
- PartitionIterable.java: seperate -> separate
- PartitionTree.java: bofore -> before
- StatsPublisher.java: statics -> statistics (Javadoc)
- VariantBuilder.java: calcualtion -> calculation
- HiveMaterializedViewsRegistry.java: partion -> partition
- HiveUtils.java: characeters -> characters
- BasicStatsNoJobTask.java: intresting -> interesting (exception message)
- debug/Utils.java: directoty -> directory
- HiveClientCache.java: wont -> won't
- metastore-validation-test.sh: occurances -> occurrences,
  retrive -> retrieve (x4), begining -> beginning,
  detemine -> determine, thrid -> third (x2)
  (comment text only; the num_occurances() function name is unchanged)

No functional changes.
@sonarqubecloud

Copy link
Copy Markdown

@Akanksha-kedia

Copy link
Copy Markdown
Author

cc @abstractdog — saw you're actively reviewing/merging Hive PRs; this is a small comment/doc-only typo fix, no JIRA filed since it's non-functional (happy to file one if preferred). Would appreciate a look when you have time.

@abstractdog

Copy link
Copy Markdown
Contributor

cc @abstractdog — saw you're actively reviewing/merging Hive PRs; this is a small comment/doc-only typo fix, no JIRA filed since it's non-functional (happy to file one if preferred). Would appreciate a look when you have time.

@Akanksha-kedia these kind of PRs are always welcome, thanks!
please create a JIRA, it's a must-have regardless of the actual functionality change or not

@Akanksha-kedia

Akanksha-kedia commented Aug 31, 2026 via email

Copy link
Copy Markdown
Author

@abstractdog

Copy link
Copy Markdown
Contributor

Can u give me link where i can create jira id ?

On Mon, 31 Aug 2026 at 3:41 PM, Bodor Laszlo @.> wrote: abstractdog left a comment (apache/hive#6742) <#6742 (comment)> cc @abstractdog https://github.com/abstractdog — saw you're actively reviewing/merging Hive PRs; this is a small comment/doc-only typo fix, no JIRA filed since it's non-functional (happy to file one if preferred). Would appreciate a look when you have time. @Akanksha-kedia https://github.com/Akanksha-kedia these kind of PRs are always welcome, thanks! please create a JIRA, it's a must-have regardless of the actual functionality change or not — Reply to this email directly, view it on GitHub <#6742?email_source=notifications&email_token=AVL2AZWRBTODXPPEN6IJCKD5MVFMBA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKNBXGY4DKNZYGM42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-5476857839>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVL2AZWCBQCCDNGHWF3CMQ35MVFMBAVCNFSNUABCKJSXA33TNF2G64TZHMZDANRUGQ2DWSLTON2WKOZVGI4TQMZYGMZTKMVBOYBA . You are receiving this because you were mentioned.Message ID: @.>

yeah, please use the asf jira for this, HIVE project: https://issues.apache.org/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants