You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "h-vetinari (via GitHub)" <gi...@apache.org> on 2023/04/27 07:20:27 UTC
[GitHub] [arrow] h-vetinari opened a new issue, #35350: BUG: failing test with grpc 1.54
h-vetinari opened a new issue, #35350:
URL: https://github.com/apache/arrow/issues/35350
### Describe the bug, including details regarding any error messages, version, and platform.
We're trying to [migrate](https://github.com/conda-forge/arrow-cpp-feedstock/pull/1027) to grpc 1.54 in conda-forge and running into the following test failure:
```
=================================== FAILURES ===================================
_____________________________ test_generic_options _____________________________
@pytest.mark.requires_testing_data
def test_generic_options():
"""Test setting generic client options."""
certs = example_tls_certs()
with ConstantFlightServer(tls_certificates=certs["certificates"]) as s:
# Try setting a string argument that will make requests fail
options = [("grpc.ssl_target_name_override", "fakehostname")]
client = flight.connect(('localhost', s.port),
tls_root_certs=certs["root_cert"],
generic_options=options)
with pytest.raises(flight.FlightUnavailableError):
client.do_get(flight.Ticket(b'ints'))
client.close()
# Try setting an int argument that will make requests fail
options = [("grpc.max_receive_message_length", 32)]
client = flight.connect(('localhost', s.port),
tls_root_certs=certs["root_cert"],
generic_options=options)
with pytest.raises(pa.ArrowInvalid):
> client.do_get(flight.Ticket(b'ints'))
test_flight.py:1976:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
pyarrow/_flight.pyx:1539: in pyarrow._flight.FlightClient.do_get
???
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
> ???
E pyarrow._flight.FlightCancelledError: Flight cancelled call, with message: CANCELLED. gRPC client debug context: UNKNOWN:Error received from peer {grpc_message:"CANCELLED", grpc_status:1, created_time:"2023-04-27T01:41:27.937716704+00:00"}. Client context: IOError: Server never sent a data message. Detail: Internal
pyarrow/_flight.pyx:65: FlightCancelledError
----------------------------- Captured stderr call -----------------------------
E0427 01:41:27.922066694 16706 ssl_transport_security.cc:1854] No match found for server name: fakehostname.
```
This also happens for arrow [10.x](https://github.com/conda-forge/arrow-cpp-feedstock/pull/1026); I presume it's the same for 9.x and 8.x, though we're not running the full test suite on those versions.
Not sure if this is just a compatibility issue (e.g. something changed within grpc), or a bug, or ...
### Component(s)
FlightRPC, Packaging
--
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: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] h-vetinari closed issue #35350: BUG: failing test with grpc 1.54
Posted by "h-vetinari (via GitHub)" <gi...@apache.org>.
h-vetinari closed issue #35350: BUG: failing test with grpc 1.54
URL: https://github.com/apache/arrow/issues/35350
--
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: issues-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] lidavidm commented on issue #35350: BUG: failing test with grpc 1.54
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #35350:
URL: https://github.com/apache/arrow/issues/35350#issuecomment-1524956631
I think we already ran into that and fixed it:
https://github.com/apache/arrow/blob/07d02d6ccaf2521cba102ac910c4def97cd3610c/python/pyarrow/tests/test_flight.py#L2048
Is it possible to skip this in conda-forge for now? The next release would pick up this fix.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] h-vetinari commented on issue #35350: BUG: failing test with grpc 1.54
Posted by "h-vetinari (via GitHub)" <gi...@apache.org>.
h-vetinari commented on issue #35350:
URL: https://github.com/apache/arrow/issues/35350#issuecomment-1525070654
Ah great, thanks a lot. I'll just backport https://github.com/apache/arrow/commit/bce43175aa8cfb4534d3efbcc092f697f25f0f5a, that's cleaner. :)
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org