itsjunetime opened a new pull request, #13068:
URL: https://github.com/apache/datafusion/pull/13068

   ## Which issue does this PR close?
   
   This will not close, but rather just helps with the situation in, #12733.
   
   ## Rationale for this change
   
   As I was trying to fix other issues, I ran into more instances where Schema 
metadata could theoretically be dropped. These specific issues don't have 
reproducers (since I just found them while working on other stuff), but I think 
they are clearly issues nonetheless.
   
   ## What changes are included in this PR?
   
   Mostly changing `Schema::new` to `Schema::new_with_metadata` and 
`schema_builder.finish()` to `schema_builder.finish().with_metadata(...)`
   
   ## Are these changes tested?
   
   Yes, all tests still pass.
   
   ## Are there any user-facing changes?
   
   No


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to