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

Posted to github@arrow.apache.org by "icexelloss (via GitHub)" <gi...@apache.org> on 2023/04/03 22:10:11 UTC

[GitHub] [arrow] icexelloss commented on pull request #34872: GH-34871: [C++] Fixed the add_dataset_test function to properly refer to the test file.

icexelloss commented on PR #34872:
URL: https://github.com/apache/arrow/pull/34872#issuecomment-1495052187

   @westonpace I approved. 
   
   Although, I am curious how did this happen? Looking this the acero refactor PR:
   https://github.com/apache/arrow/commit/f137f29f53d2fef31cfdd9ae16e539de5f4b8de8#diff-1855e31fd38e7fa63c95344ad9992c27d2f609f015a06d45ca3dd3ecc708243d
   
   It doesn't seem to remove anything from tests. What causes the dataset test to be disabled? 


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