no23reason opened a new pull request, #48082:
URL: https://github.com/apache/arrow/pull/48082

   ### Rationale for this change
   
   After the changes in #47115, GeneratorStreams backed by anything else than 
RecordBatches failed. This includes Tables and RecordBatchReaders.
   
   This was caused by a too strict assumption that the 
RecordBatchStream#GetSchemaPayload would always get called, which is not the 
case when the GeneratorStream is backed by a Table or a RecordBatchReader.
   
   ### What changes are included in this PR?
   
   Removal of the problematic assertion and initialization of the writer object 
when it is needed first.
   
   ### Are these changes tested?
   
   Yes, via CI. Tests for the GeneratorStreams were extended so that they test 
GeneratorStreams backed by Tables and RecordBatchReaders, not just 
RecordBatches. 
   
   ### Are there any user-facing changes?
   
   No, just a fix for a regression restoring the functionality from version 
21.0.0 and earlier.
   
   * GitHub issue: https://github.com/apache/arrow/issues/48076
   


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

Reply via email to