-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Feature] Support hive catalog to read json format tables. #54884
base: main
Are you sure you want to change the base?
Conversation
xiangyi.zhu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
fe/fe-core/src/main/java/com/starrocks/connector/hive/HiveStorageFormat.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/com/starrocks/connector/hive/HiveMetastoreApiConverter.java
Outdated
Show resolved
Hide resolved
@kevincai Hi Kevin ,can you review this PR for me |
@zhuxiangyi Thanks for filing this issue. Overall LGTM. Please add a SQL test |
@DorianZheng should this PR identified as "Feature" or "Enhancement"? "Feature" PR should be associated with a github feature issue and should NOT be backport to release branch. |
[FE Incremental Coverage Report]✅ pass : 7 / 7 (100.00%) file detail
|
[Java-Extensions Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
[BE Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
|
fe/fe-core/src/main/java/com/starrocks/connector/hive/HiveMetastoreApiConverter.java
Outdated
Show resolved
Hide resolved
089d8db
to
3495689
Compare
Why I'm doing:
What I'm doing:
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check: