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.

Sign up for GitHub

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

Jump to bottom

Move HiveSchemaProperties class to presto-hive-metastore module #24401

Merged
merged 1 commit into from
Jan 22, 2025
Merged

Move HiveSchemaProperties class to presto-hive-metastore module #24401

merged 1 commit into from
Jan 22, 2025

Conversation

agrawalreetika
Copy link
Member

Description

Move HiveSchemaProperties class to presto-hive-metastore module

Motivation and Context

Clean up

Impact

Test Plan

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 ==

@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Jan 20, 2025
@prestodb-ci prestodb-ci requested review from a team, czentgr and aditi-pandit and removed request for a team January 20, 2025 20:34
@agrawalreetika agrawalreetika self-assigned this Jan 20, 2025
@agrawalreetika agrawalreetika mentioned this pull request Jan 20, 2025
Open
6 tasks
@tdcmeehan tdcmeehan merged commit 6d57079 into prestodb:master Jan 22, 2025
52 checks passed
@agrawalreetika agrawalreetika deleted the code-cleanup branch January 23, 2025 02:53
shangm2 pushed a commit to shangm2/presto that referenced this pull request Jan 23, 2025
@agrawalreetika @shangm2
Move HiveSchemaProperties class to presto-hive-metastore module (pres…
0730918 
…todb#24401)
This was referenced Jan 28, 2025
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ZacBlanco ZacBlanco ZacBlanco approved these changes

@tdcmeehan tdcmeehan tdcmeehan approved these changes

@hantangwangd hantangwangd hantangwangd approved these changes

@presto-oss presto-oss Awaiting requested review from presto-oss presto-oss is a code owner automatically assigned from prestodb/committers

@czentgr czentgr Awaiting requested review from czentgr czentgr was automatically assigned from prestodb/ibm-reviewers

@aditi-pandit aditi-pandit Awaiting requested review from aditi-pandit aditi-pandit was automatically assigned from prestodb/ibm-reviewers

Assignees

@agrawalreetika agrawalreetika

Labels
from:IBM PR from IBM
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@agrawalreetika @ZacBlanco @tdcmeehan @hantangwangd @prestodb-ci