This is an automated email from the ASF dual-hosted git repository.
yuqi1129 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/main by this push:
new aca4b45fcc [MINOR] Fix Python formatting in client and matrix script
(#11444)
aca4b45fcc is described below
commit aca4b45fcc3f7ca89b2d6ea00b6a0c272eb5df2d
Author: Jerry Shao <[email protected]>
AuthorDate: Fri Jun 5 14:15:00 2026 +0800
[MINOR] Fix Python formatting in client and matrix script (#11444)
## What changes were proposed in this pull request?
Minor Python formatting fixes:
- Remove an extra blank line in `gravitino_metalake.py`
- Reformat long function signatures in `run_lance_ray_matrix.py` to
comply with line length limits
## Why are the changes needed?
Code style/formatting cleanup.
## Does this PR introduce _any_ user-facing change?
No.
## How was this pull request tested?
No logic changes; formatting only.
---
clients/client-python/gravitino/client/gravitino_metalake.py | 1 -
clients/client-python/scripts/run_lance_ray_matrix.py | 8 ++++++--
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/clients/client-python/gravitino/client/gravitino_metalake.py
b/clients/client-python/gravitino/client/gravitino_metalake.py
index 75394a1208..beb4baf96a 100644
--- a/clients/client-python/gravitino/client/gravitino_metalake.py
+++ b/clients/client-python/gravitino/client/gravitino_metalake.py
@@ -14,7 +14,6 @@
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
-
# pylint: disable=too-many-lines
import logging
from typing import Dict, List, Optional
diff --git a/clients/client-python/scripts/run_lance_ray_matrix.py
b/clients/client-python/scripts/run_lance_ray_matrix.py
index 3a0fc94842..f2daecb59e 100644
--- a/clients/client-python/scripts/run_lance_ray_matrix.py
+++ b/clients/client-python/scripts/run_lance_ray_matrix.py
@@ -146,7 +146,9 @@ def ensure_venv(python: str, venv_dir: Path) -> Path:
return venv_python
-def _deps_sentinel(venv_dir: Path, version: str, ray_spec: str,
lance_namespace_spec: str) -> Path:
+def _deps_sentinel(
+ venv_dir: Path, version: str, ray_spec: str, lance_namespace_spec: str
+) -> Path:
"""Return the path to the sentinel file that marks a fully-installed venv.
The sentinel encodes all dep specs so any change in pinned versions forces
@@ -171,7 +173,9 @@ def install_deps(
"""
sentinel = _deps_sentinel(venv_dir, version, ray_spec,
lance_namespace_spec)
if sentinel.exists():
- print(f"[matrix] venv for lance-ray=={version} already populated,
skipping pip install")
+ print(
+ f"[matrix] venv for lance-ray=={version} already populated,
skipping pip install"
+ )
return
rc = run(
