You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2024/01/04 14:22:55 UTC

[PR] Minor: Improve `PruningPredicate` docstrings more [arrow-datafusion]

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

   ## Which issue does this PR close?
   
   Related to https://github.com/apache/arrow-datafusion/issues/7869
   
   ## Rationale for this change
   
   As we have extended the PruningPredicate's features and scope, we should also improve its documentation to match.
   
   ## What changes are included in this PR?
   
   Improve the PruningPredicate doc strings to reflect its ongoing changes
   
   ## Are these changes tested?
   via doctests
   
   ## Are there any user-facing changes?
   
   Doc strings only. No code changes


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


Re: [PR] Minor: Improve `PruningPredicate` docstrings more [arrow-datafusion]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on PR #8748:
URL: https://github.com/apache/arrow-datafusion/pull/8748#issuecomment-1878851566

   > 👍 Nice write up.
   
   Thank you @Ted-Jiang !


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


Re: [PR] Minor: Improve `PruningPredicate` docstrings [arrow-datafusion]

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


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


Re: [PR] Minor: Improve `PruningPredicate` docstrings more [arrow-datafusion]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on PR #8748:
URL: https://github.com/apache/arrow-datafusion/pull/8748#issuecomment-1877178400

   fyi @my-vegetable-has-exploded and @yahoNanJing 


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