[Subtask] Refactoring SupportsCatalogs.listCatalogs() method in client side #3702
Comments
shaofengshi
added a commit
to shaofengshi/gravitino
that referenced
this issue
Jun 4, 2024
jerryshao
pushed a commit
that referenced
this issue
Jun 5, 2024
…hod to return String[] (#3741) ### What changes were proposed in this pull request? Currently, the SupportsCatalogs.listCatalogs() method returns an array of NameIdentifier to represents the catalogs. Actually a String[] will be more clear and easier to use. ### Why are the changes needed? To make the API more clear to use. Fix: #3702 ### Does this PR introduce _any_ user-facing change? Almost not; Just return type changed, will be more simple. ### How was this patch tested? Yes, many test cases cover this API.
diqiu50
pushed a commit
to diqiu50/gravitino
that referenced
this issue
Jun 13, 2024
…() method to return String[] (apache#3741) ### What changes were proposed in this pull request? Currently, the SupportsCatalogs.listCatalogs() method returns an array of NameIdentifier to represents the catalogs. Actually a String[] will be more clear and easier to use. ### Why are the changes needed? To make the API more clear to use. Fix: apache#3702 ### Does this PR introduce _any_ user-facing change? Almost not; Just return type changed, will be more simple. ### How was this patch tested? Yes, many test cases cover this API.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the subtask
Currently, the SupportsCatalogs.listCatalogs() method returns an array of NameIdentifier to represents the catalogs. Actually a String[] will be more clear and easier to use.
Parent issue
#3626
The text was updated successfully, but these errors were encountered: