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

[Python][FlightRPC] pyarrow client do_put close method after write_table did not throw flight error #29224

Closed
asfimport opened this issue Aug 6, 2021 · 1 comment

Comments

@asfimport
Copy link

hi, when I implentend flight server to do_put data, and throw exceptions proactively, and client will get OSError when close writer, but I expect FlightError.

   image-2021-08-06-17-19-27-413.png

//代码占位符

def do_put(self, context, descriptor, reader, writer):
    table = reader.read_all()
    ....
    raise fl.FlightUnavailableError()
    ....
//代码占位符
df = ...
table = pa.Table.from_pandas(df)
client = fl.connect('...')
writer, _ = client.do_put(descriptor, table.schema, options=options)
writer.write_table(table)
write.close()

also, when I throw exception before read_all, the write_table throw OSError.

image-2021-08-06-17-21-12-181.png

 

and i guess,  the FlightStreamWriter   should add write_table and close method, and check_flight_status  according to write_batch had checked. 

thx.

 

 

Reporter: lixiang li / @lilixiang
Assignee: lixiang li / @lilixiang

Original Issue Attachments:

PRs and other links:

Note: This issue was originally created as ARROW-13577. Please see the migration documentation for further details.

@asfimport
Copy link
Author

David Li / @lidavidm:
Issue resolved by pull request 10893
#10893

@asfimport asfimport added this to the 6.0.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant