laserninja opened a new pull request, #12861:
URL: https://github.com/apache/gravitino/pull/12861

   ### What changes were proposed in this pull request?
   
   This PR reformats four files under `clients/client-python` with the `black` 
version pinned in `requirements-dev.txt`. The changes are line wrapping only; 
no behavior is affected.
   
   - `gravitino/api/catalog_change.py`
   - `gravitino/api/file/fileset_change.py`
   - `tests/integration/test_model_catalog.py`
   - `tests/integration/test_relational_catalog.py`
   
   ### Why are the changes needed?
   
   These files are not formatted the way the pinned `black` version formats 
them, so the next unrelated change that touches them picks up incidental 
reformatting in its diff.
   
   The drift is not caught today because the Gradle `pythonFormat` task runs 
`black` in place rather than in check mode, so CI stays green while the 
difference accumulates. Reformatting now keeps future diffs focused on their 
actual changes.
   
   I noticed this while working on the Semantic Model Python client sub-tasks 
and kept it out of those PRs.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Reformatted with the pinned `black` version and confirmed a second run is a 
no-op, then ran the unit tests.
   
   ```
   ./gradlew :clients:client-python:test
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to