[Improvement] Possible resource leak in BaseCatalog.java

[justinmclean]

What would you like to be improved?

This code:
BaseAuthorization<?> authorization =
BaseAuthorization.createAuthorization(classLoader, authorizationProvider);

This probably needs to be closed if an exception occurs?

How should we improve?

No response

[Brijeshthummar02]

@justinmclean can u provide path to the file i will fix it!

[justinmclean]

core/src/main/java/org/apache/gravitino/connector/BaseCatalog.java

[justinmclean]

@Brijeshthummar02 I saw you closed your PR. Do you still want to work on this issue?

[Brijeshthummar02]

@justinmclean yes i am trying to make new clone and fix it, when i deleted the fork it by default closed PR. I am working on it.

[justinmclean]

Excellent, if you run into any issues I'll try to help.

[Brijeshthummar02]

@justinmclean there are multiple BaseCatalog.java provide the file to path

[justinmclean]

The one you want is core/src/main/java/org/apache/gravitino/connector/BaseCatalog.java