hive icon indicating copy to clipboard operation
hive copied to clipboard

HIVE-28994: Implement direct sql for getAllDatabases and getDatabases in RawStore

Open shuyouZZ opened this issue 8 months ago • 5 comments

What changes were proposed in this pull request?

Implement direct sql for getAllDatabases and getDatabases in RawStore.

Why are the changes needed?

Improve the performance of getAllDatabases and getDatabases.

Does this PR introduce any user-facing change?

No

How was this patch tested?

mvn test -Dtest.groups= -Dtest=org.apache.hadoop.hive.metastore.TestObjectStore#testDatabaseOps -pl :hive-standalone-metastore-server

shuyouZZ avatar Jun 10 '25 03:06 shuyouZZ

CI failed probably because of this change.

okumin avatar Jun 16 '25 10:06 okumin

CI failed probably because of this change.

I can not open the continuous-integration page, it shows Not Found. The previous execution of CI was successful.

shuyouZZ avatar Jun 16 '25 12:06 shuyouZZ

@shuyouZZ You can sign in the Jenkins with a GitHub account

okumin avatar Jun 16 '25 13:06 okumin

@shuyouZZ You can sign in the Jenkins with a GitHub account

thanks, let me try

shuyouZZ avatar Jun 17 '25 03:06 shuyouZZ

is that PR abandoned?

deniskuzZ avatar Aug 07 '25 13:08 deniskuzZ

is that PR abandoned?

Sorry for taking so long to reply. I still hope to continue working on this PR, but I'm not sure why this build failed.

shuyouZZ avatar Aug 08 '25 13:08 shuyouZZ

@shuyouZZ Although this might not be the direct reason, you have to rebase the topic branch. The CI says it was tested with JDK 17, but the upstream uses JDK 21 https://github.com/apache/hive/actions/runs/15722082143/job/44304757814?pr=5852

okumin avatar Aug 09 '25 04:08 okumin

@shuyouZZ Although this might not be the direct reason, you have to rebase the topic branch. The CI says it was tested with JDK 17, but the upstream uses JDK 21 https://github.com/apache/hive/actions/runs/15722082143/job/44304757814?pr=5852

Ok. Let me try force rebase.

shuyouZZ avatar Aug 09 '25 15:08 shuyouZZ

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Feel free to reach out on the [email protected] list if the patch is in need of reviews.

github-actions[bot] avatar Oct 11 '25 00:10 github-actions[bot]