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

959 stars 302 forks source link

[Subtask] Fix broad-exception-caught Pylint rule for client-python #3766

Closed noidname01 closed 1 month ago

noidname01 commented 4 months ago

Describe the subtask

Currently the try...except block only catchs general exception Exception, please replace it with more specific exceptions.

Parent issue

3560

rich7420 commented 3 months ago

I’m interested in working on this issue. Can you please provide more details about this issue?

noidname01 commented 3 months ago

Hi @rich7420, thanks for your interest for this issue.

You can try to remove the broad-exception-caught rule from disable section in pylintrc and run the pylint task to run pylint checks, you would see where the problems are, then you can try to solve them.

noidname01 commented 3 months ago

Actually this issue is related to the current issue I make: #4012

The current client-python should have better error handling structure like client-java, maybe you can help to solve this issue first?