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

   ## Which issue does this PR close?
   
   None.
   
   ## Rationale for this change
   
   All `register_*` methods on `SessionContext` should have similar signatures 
for the sake of API consistency.
   
   ## What changes are included in this PR?
   
   Update `SessionContext::register_batch` and `SessionContext::register_arrow` 
to have similar signatures than the rest of `SessionContext::register_*` 
methods.
   
   ## Are these changes tested?
   
   I don't think any tests are required for this. Type checking done by the 
compiler should be enough. Let me know if you disagree.
   
   ## Are there any user-facing changes?
   
   I don't think so. All previous calls to these methods should still work as 
before.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to