You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "tv42 (via GitHub)" <gi...@apache.org> on 2023/09/19 22:09:39 UTC

[GitHub] [arrow-datafusion] tv42 opened a new pull request, #7603: Expose contents of Constraints

tv42 opened a new pull request, #7603:
URL: https://github.com/apache/arrow-datafusion/pull/7603

   ## Which issue does this PR close?
   
   Closes #7511
   
   ## Rationale for this change
   
   Make the whole LogicalPlan introspectable again.
   
   ## What changes are included in this PR?
   
   <!--
   There is no need to duplicate the description in the issue here but it is sometimes worth providing a summary of the individual changes in this PR.
   -->
   
   ## Are these changes tested?
   
   Simple unit test included.
   
   ## Are there any user-facing changes?
   
   Publishes `Constraint`.
   
   Implements `Deref` for `Constraints` to get `&[Constraint]`. This allows iterating `Constraints` (Without having to implement a whole iterator, or using `impl Iterator`; code is easy to switch to either of those approaches is someone has a very strong opinion. KISS sounds right to me.).


-- 
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-unsubscribe@arrow.apache.org

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


[GitHub] [arrow-datafusion] alamb merged pull request #7603: Expose contents of Constraints

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb merged PR #7603:
URL: https://github.com/apache/arrow-datafusion/pull/7603


-- 
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-unsubscribe@arrow.apache.org

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