alamb commented on issue #8329: URL: https://github.com/apache/arrow-datafusion/issues/8329#issuecomment-1829983162
I also want to make my algorithm for prioritizing PR reviews more visible, so adding it here (and will add to descriptions too). I want to point out we have many [committers](https://arrow.apache.org/committers/) who can review and merge PRs as well, so this is not the priorities of the project as a whole, just the approximate algorithm I am using Priority: 1. Bug fixes (where something is just incorrect), especially regressions (where it used to work and now does not) 2. Improvements directly related to stuff needed for InfluxDB (as I am employed by them) 3. PRs for documentation and test improvements (I view these as very strategically important) 4. PRs that I think are strategically important 5. Other new features / additions to functionality The current strategically important projects I am tracking are: * https://github.com/apache/arrow-datafusion/issues/8227 * https://github.com/apache/arrow-datafusion/issues/7926 * https://github.com/apache/arrow-datafusion/issues/8045 Note that adding new array functionality (e.g. https://github.com/apache/arrow-datafusion/issues/6980 and its children) is very low on my priority list, not because I don't think they are valuable, but because I think they are less valuable than the other projects going on Thus, if you are interested in contributing to DataFusion and are interested in a fast turn around time I would recommend looking into bug fixes / test improvements. If you propose adding new functionality, the review cycle will be longer. You can make it a shorter cycle by looking at the comments on other recent PRs and following the same model (e.g. ensure there are tests in sqllogictest for example) -- 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]
