Skip to content

[python] Fix the flake8 failure in jindo_file_system_test.py - #10550

Closed
sundapeng wants to merge 1 commit into
apache:masterfrom
sundapeng:fix-jindo-test-flake8
Closed

sundapeng wants to merge 1 commit into
apache:masterfrom
sundapeng:fix-jindo-test-flake8

Conversation

@sundapeng

Copy link
Copy Markdown
Member

Purpose

Since #10536, lint-python.sh fails flake8:

./pypaimon/tests/jindo_file_system_test.py:407:36: E127 continuation line over-indented for visual indent

Every Python test job of #10536's own run failed on this line (for example job 115523084822). Pull request CI checks the merge with master, so every Python pull request now fails the same way. The latest finished master run (38416675963 on 46426e8) stopped earlier at the RAT check, which #10548 has since fixed, so master has not reached flake8 yet.

This pull request aligns the continuation line with the opening parenthesis. No behavior change.

Tests

  • flake8 7.4.1 with dev/cfg.ini over the file list lint-python.sh builds reports nothing; on master it reports the line above.
  • pytest pypaimon/tests/jindo_file_system_test.py: 15 passed, 19 skipped (no OSS bucket configured).

apache#10536 left a continuation line over-indented (E127), so lint-python.sh
fails flake8 on master and on every Python pull request. Align it with the
opening parenthesis.
@sundapeng

Copy link
Copy Markdown
Member Author

Closing as superseded by #10552: it merged the same change to jindo_file_system_test.py (8e9fee6), so this branch no longer differs from master.

@sundapeng sundapeng closed this Oct 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant