kou commented on PR #38256: URL: https://github.com/apache/arrow/pull/38256#issuecomment-1767400070
We can use either approaches ((1) one PR that includes `.proto` change, 2 implementations and integration tests or (2) 3 PRs for (2.1) `.proto`, (2.2) 1 implementation and integration tests and (2.3) another implementation and integration tests). We don't have a requirement for this. See also: https://arrow.apache.org/docs/format/Changing.html As you said, the latter will be smooth to review. The former may be easy to implement because the PR author can work on only one branch. (We may find a `.proto` problem when we implement the feature and integration tests. In the case, we need to fix `.proto` and 2 implementations in one branch or three branches.) -- 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]
