You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/03/24 05:31:19 UTC

[GitHub] [arrow] vibhatha commented on pull request #12664: ARROW-15658: [C++] Parquet pushdown filtering fails if the filter expression uses numeric field references

vibhatha commented on pull request #12664:
URL: https://github.com/apache/arrow/pull/12664#issuecomment-1077198537


   @lidavidm I updated the test cases and added a check for the nested field paths. To add an error checking test, I think some of the util functions need to be changed to grab a returning `NotImplemented` status. The `SetFilter` function call triggers the error. Is it okay to return a `Status` from that method. I didn't do the change but would like to know if it is okay to do. I can proceed with it if it is okay. But is there a better alternative here? 


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