mapleFU commented on PR #38885:
URL: https://github.com/apache/arrow/pull/38885#issuecomment-1854195702
```c++
Future<> WriteNext(std::shared_ptr<RecordBatch> next) {
// May want to prototype / measure someday pushing the async write down
further
return DeferNotOk(options_.filesystem->io_context().executor()->Submit(
[self = this, batch = std::move(next)]() {
int64_t rows_to_release = batch->num_rows();
Status status = self->writer_->Write(batch);
self->writer_state_->rows_in_flight_throttle.Release(rows_to_release);
return status;
}));
}
```
`self->writer_state_->rows_in_flight_throttle.Release` might trigger the
callback, I guess this might trigger the problem. But I need some time to
convince @kou
--
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]
