raulcd opened a new issue, #38262:
URL: https://github.com/apache/arrow/issues/38262

   ### Describe the enhancement requested
   
   Follow up from: https://github.com/apache/arrow/issues/38239
   
   We decided to deactivate errors on Warnings in order to have a passing CI 
for the Release and this is a follow up in order to re-enable them. The issue 
we faced can be seen here : https://github.com/apache/arrow/pull/38238 and 
here: https://github.com/apache/arrow/pull/38259
   
   There seems to be an issue if we try to have an ignore quote that contains 
spaces, see:
   ```
   $ export PYTEST_ARGS="-W error -W 'ignore:Setting custom 
ClientSession:DeprecationWarning'"
   $ PYTHONASYNCIODEBUG=1 pytest 
python/pyarrow/tests/test_dataset.py::test_open_dataset_from_uri_s3_fsspec 
$PYTEST_ARGS
   ERROR: while parsing the following warning configuration:
   
     'ignore:Setting
   
   This error occurred:
   
   invalid action: "'ignore"
   ```
   We tried the approach of removing `-W error` from `PYTEST_ARGS` and changing 
`setup.cfg` to contain:
   ```
   [tool:pytest]
   addopts = --ignore=scripts
   filterwarnings =
       error
       # https://github.com/apache/arrow/issues/38239
       ignore:Setting custom ClientSession:DeprecationWarning
   ```
   but this configuration doesn't seem to be used as it did not fail if 
removing the `ignore, as [suggested 
here](https://github.com/apache/arrow/pull/38259#issuecomment-1761344012) this 
is probably because we run the installed tests, not in-place (using `--pyargs 
pyarrow`)
   
   If we try to use the `setup.cfg` we probably want to add some jobs with 
`PYTEST_ARGS: -W always`
   
   ### Component(s)
   
   Continuous Integration, Python


-- 
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