Skip to content
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

Added Hive Metastore documentation in the Hive Connector, including a… #24549

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

alexaustin007
Copy link

@alexaustin007 alexaustin007 commented Feb 13, 2025

…dditional metastore resource links for the ease of new users.

Description

Changes in the documentation of hive metastore

Motivation and Context

For the ease of new users to understand hive metastore

Yes, it fixes an open issue, #24523 (comment)

Impact

No impact

Test Plan

Built and Tested locally in the index page

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== NO RELEASE NOTE ==

…dditional metastore resource links for the ease of new users.
Copy link

linux-foundation-easycla bot commented Feb 13, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: alexaustin007 / name: Alex Austin Chettiar (e7a58c7, 3b74473)

Copy link
Contributor

@steveburnett steveburnett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the documentation! This does address the suggested content mentioned in the discussion of #24511. Pull branch, local doc build, verified all the links work. Nice job!

Found a nit of phrasing, everything else looks good at this point.

presto-docs/src/main/sphinx/connector/hive.rst Outdated Show resolved Hide resolved
@steveburnett
Copy link
Contributor

@agrawalreetika, @imjalpreet - if you have time, would you look at this PR that addresses the discussion of #24511?

@imjalpreet, you mentioned in that discussion that there would be a few more topics you'd like to see linked to or mentioned in this new section: please let @alexaustin007 know what those additional topics are when you can.

@steveburnett
Copy link
Contributor

When you have time, please update the Release Notes section in this PR to

== NO RELEASE NOTE ==

by deleting the other lines in the Release Notes section.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🏗 In progress
Development

Successfully merging this pull request may close these issues.

2 participants