The GitHub Actions job "Fory CI" on fory.git/main has succeeded.
Run started by GitHub user chaokunyang (triggered by chaokunyang).

Head commit for run:
f26681c1598aaa57051fbf1480180ce2e84cc57f / Emre Şafak 
<[email protected]>
fix: Correct pyarrow installation check and add bazel troubleshooting (#2418)

I made a mistake in #2396 when I changed

```python
func = pytest.mark.fory_format(func)
arrow_installed = False
if pa is not None and hasattr(pa, "get_library_dirs"):
    arrow_installed = True
func = pytest.mark.skipif(not arrow_installed, reason="pyarrow not 
installed")(func)
return func
```
to
```python
arrow_not_installed = pa is None or hasattr(pa, "get_library_dirs")
mark_decorator = pytest.mark.skipif(arrow_not_installed, reason="pyarrow not 
installed")(func)
return mark_decorator

```

The second condition, `hasattr(pa, "get_library_dirs")`, should have
been negated. I noticed this when [arrow tests were incorrectly being
skipped](https://github.com/apache/fory/actions/runs/16297878100/job/46024343029)
:|

## What does this PR do?

* Fix incorrect pyarrow installation check in core.py
* Add troubleshooting steps for bazel caching issues in README.md

Co-authored-by: Emre Şafak <[email protected]>

Report URL: https://github.com/apache/fory/actions/runs/16309342231

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to