apache / apache/gravitino

[Improvement] Fix client-python codes to conform Pylint Rules

Open
#3,560 2 comments 0 reactions 0 assignees View on GitHub
good first issue improvement
Dominant language
Java
Stars
3.2k
Forks
935
Avg merge
1d 15h
Merged PRs (30d)
315

Description

### What would you like to be improved?

There are some Pylint Rules that were disabled now (labeled by `TODO-fix` in `pylintrc`) but needed to be fixed to conform the Pylint standards.

- [ ] missing-timeout #3765
- [x] redefined-builtin: #3561
- [ ] broad-exception-caught #3766
- [ ] duplicate-code #3767
- [x] method-hidden #3768
- [x] invalid-field-call #3769
- [x] protected-access #3770
- [ ] too-many-arguments
- [x] no-name-in-module #3771
- [x] inconsistent-return-statements #3771

### How should we improve?

_No response_

Contributor guide

Open the contributing guide

Research direction

Start by reading `pylintrc` and the remaining unchecked rules in the issue, then review the corresponding code in `client-python`. The issue is done when the listed remaining rules no longer need to be disabled and the client conforms to them.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
devtools
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.