Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[C++][Python][FlightRPC] FlightServer init error with ipv6 location #37539

Closed
tianjiqx opened this issue Sep 4, 2023 · 0 comments · Fixed by #37552
Closed

[C++][Python][FlightRPC] FlightServer init error with ipv6 location #37539

tianjiqx opened this issue Sep 4, 2023 · 0 comments · Fixed by #37552

Comments

@tianjiqx
Copy link

tianjiqx commented Sep 4, 2023

Describe the bug, including details regarding any error messages, version, and platform.

version: pyarrow 13.0.0

error stack:

location: Location = Location.for_grpc_tcp(host="[::]", port=5005)
server = FlightServer(args.host, location,
                      tls_certificates=tls_certificates,
                      verify_client=args.verify_client)


    super(FlightServer, self).__init__(
  File "pyarrow/_flight.pyx", line 2765, in pyarrow._flight.FlightServerBase.__init__
  File "pyarrow/_flight.pyx", line 2838, in pyarrow._flight.FlightServerBase.init
  File "pyarrow/_flight.pyx", line 78, in pyarrow._flight.check_flight_status
  File "pyarrow/error.pxi", line 100, in pyarrow.lib.check_status
pyarrow.lib.ArrowInvalid: Cannot parse URI: 'grpc+tcp://:::5005'

I see that it has been fixed in #8585, but it still seems to be unresolved in pyarrow, or is there a mistake in the way I use it?

The started service will listen to ipv6 and can support ipv6 client requests. But it cannot support listening to the specified ip

netstat -tlnp | grep 5005
tcp6       0      0 :::5005                 :::*                    LISTEN      853314/python


client work with ipv6: [::1]:5005 

Component(s)

Python

@lidavidm lidavidm changed the title [Python][FlightRPC] FlightServer init error with ipv6 location [C++][Python][FlightRPC] FlightServer init error with ipv6 location Sep 4, 2023
@lidavidm lidavidm self-assigned this Sep 4, 2023
lidavidm added a commit to lidavidm/arrow that referenced this issue Sep 4, 2023
@kou kou closed this as completed in #37552 Sep 5, 2023
kou pushed a commit that referenced this issue Sep 5, 2023
### Rationale for this change

Binding to an IPv6 address got broken along the way

### What changes are included in this PR?

Fix the binding and see if we can enable IPv6 testing in CI

### Are these changes tested?

If CI permits

### Are there any user-facing changes?

No
* Closes: #37539

Authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou kou added this to the 14.0.0 milestone Sep 5, 2023
loicalleyne pushed a commit to loicalleyne/arrow that referenced this issue Nov 13, 2023
…e#37552)

### Rationale for this change

Binding to an IPv6 address got broken along the way

### What changes are included in this PR?

Fix the binding and see if we can enable IPv6 testing in CI

### Are these changes tested?

If CI permits

### Are there any user-facing changes?

No
* Closes: apache#37539

Authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
…e#37552)

### Rationale for this change

Binding to an IPv6 address got broken along the way

### What changes are included in this PR?

Fix the binding and see if we can enable IPv6 testing in CI

### Are these changes tested?

If CI permits

### Are there any user-facing changes?

No
* Closes: apache#37539

Authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants