search

apache / gravitino

World's most powerful open data catalog for building a high-performance, geo-distributed and federated metadata lake.
https://gravitino.apache.org
Apache License 2.0
1.09k stars 343 forks source link

[Improvement] GravitinoClient needs metalake to obtain version #5051

Open justinmclean opened 1 month ago

justinmclean commented 1 month ago

What would you like to be improved?

Currently the GravitinoClient requires a known metalake name if you want to get the server version number. This is an issue if no metalakes exist.

How should we improve?

You should be able to get the version number without knowing the name of any existing metalakes.

lsyulong commented 1 month ago

Hello, I would like to give it a try. Can this task be assigned to me

lsyulong commented 1 month ago

Where should I have made the modifications? Can you provide me with assistance

justinmclean commented 1 month ago

You want to take a look at GravitinoClient.java / GravitinoClientBase.java and the serverVersion() method. serverVersion() doesn't require a metalake, but the GravitinoClient build method checks that the metalake is not null or empty. The GravitinoClient constructor also calls loadMetalake.

lsyulong commented 1 month ago

You want to take a look at GravitinoClient.java / GravitinoClientBase.java and the serverVersion() method. serverVersion() doesn't require a metalake, but the GravitinoClient build method checks that the metalake is not null or empty. The GravitinoClient constructor also calls loadMetalake.

Hello, I have made some modifications based on your prompt. Do you have any questions? Feel free to communicate with me anytime

YuanG0319tt commented 1 month ago

Hey, was that possible that I can work with this task together?