nealrichardson commented on PR #13541:
URL: https://github.com/apache/arrow/pull/13541#issuecomment-1192547994

   > > @dragosmg check out [#13397 
(comment)](https://github.com/apache/arrow/pull/13397#discussion_r926785889). I 
think this explains why "the filter node is gone" on the head queries--it gets 
evaluated 🙀 inside of Build(). For now we should probably not try to show the 
query `if (is_collapsed(.data) && has_head_tail(.data$.data))` because that 
will cause the inner query to evaluate. We can back that out after 
[ARROW-16628](https://issues.apache.org/jira/browse/ARROW-16628), I believe 
(should add a comment pointing to that issue too).
   > 
   > Not sure adding a comment referencing 
[ARROW-16628](https://issues.apache.org/jira/browse/ARROW-16628) is needed 
since I am no longer touching the R6 `$Build()` method. I added a comment on 
[ARROW-17184](https://issues.apache.org/jira/browse/ARROW-17184).
   
   But you're still calling $Build, and in this case, the inner query will be 
evaluated inside of that, so we should check before calling $Build.


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