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

Remove duplicate getParquetType method from ParquetPageSourceFactory #16699

Merged
merged 1 commit into from
Mar 25, 2023

Conversation

leetcode-1533
Copy link
Contributor

Description

Remove duplicate getParquetType method from ParquetPageSourceFactory

Additional context and related issues

Release notes

(x) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot
Copy link

cla-bot bot commented Mar 23, 2023

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: yluan.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@cla-bot cla-bot bot added the cla-signed label Mar 23, 2023
@leetcode-1533 leetcode-1533 marked this pull request as ready for review March 23, 2023 18:05
@github-actions github-actions bot added hive Hive connector tests:hive labels Mar 23, 2023
@leetcode-1533
Copy link
Contributor Author

hi, updated, let me know if there is more issues!

@raunaqmorarka raunaqmorarka merged commit 86c609f into trinodb:master Mar 25, 2023
@github-actions github-actions bot added this to the 411 milestone Mar 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed hive Hive connector
Development

Successfully merging this pull request may close these issues.

2 participants