AlenkaF commented on PR #46330: URL: https://github.com/apache/arrow/pull/46330#issuecomment-2853820168
I agree that the conditional behavior is worth documenting. > If the dataset module is enabled a list of strings can also be used which correspond to a list of files. This is great. I would just suggest changing "can also be used" to "can also be passed" to align more closely with the existing phrasing. One more thing to note: this will require a test that would use the dataset mark. Because of that, the PR no longer fits into the MINOR category and will need a corresponding issue to track the change. -- 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-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
