[I] [C++] DatasetWriter cleanup resource when not able to write [arrow]

["mapleFU (via GitHub)" <gi...@apache.org>]

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

   ### Describe the enhancement requested
   
   ```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;
           }));
     }
   ```
   
   The `DatasetWriterFileQueue::WriteNext` will write-batch and Release the chunk-size. However, if the file-size is too large, it will failed to release.
   
   ### Component(s)
   
   C++


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

Re: [I] [C++] DatasetWriter cleanup resource when not able to write [arrow]

["kou (via GitHub)" <gi...@apache.org>]

kou closed issue #38884: [C++] DatasetWriter cleanup resource when not able to write
URL: https://github.com/apache/arrow/issues/38884


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