[#6513] fix resource leak in BaseCatalog

[yunchipang]

What changes were proposed in this pull request?

Use a try-with-resources pattern to ensure the BaseAuthorization object is automatically closed when execution exits the try block, regardless of whether it completes normally or due to an exception.

Why are the changes needed?

To prevent memory leak.

Fix: #6513

Does this PR introduce any user-facing change?

No.

How was this patch tested?

(Please test your changes, and provide instructions on how to test it:

1. If you add a feature or fix a bug, add a test to cover your changes.
2. If you fix a flaky test, repeat it for many times to prove it works.)

[jerqi]

Like last discussion, the Authorization is like a factory class. It's a little weird to have the method close.

[yunchipang]

@jerqi thanks for reviewing. I think i was not involved in the discussion. Are we closing the issue then? Feel free to close #6513 if it's no longer valid.

[xunliu]

@jerryshao What do you think?