I think it's worthwhile enough to justify the work for the patch. If we do end up doing the patch, then we should also include this [1] change for the Go side which, while significant, I didn't believe to be significant enough to warrant a patch on its own. But it is definitely a good idea to include this in a patch release if we're going to be doing one for other reasons.
--Matt [1]: https://github.com/apache/arrow/issues/35337 On Thu, May 18, 2023 at 1:18 PM Ian Cook <i...@ursacomputing.com> wrote: > There is also a major issue with the 12.0.0 R package that has now > been fixed in the repo [2] and needs to be resubmitted to CRAN soon. > The R package developers are supportive of a 12.0.1 patch release > happening soon so that the resubmission of the R package to CRAN can > also include the fix for the performance regression you mention. > > Ian > > [2] https://github.com/apache/arrow/pull/35612 > > On Thu, May 18, 2023 at 1:04 PM Weston Pace <weston.p...@gmail.com> wrote: > > > > Regrettabl, 12.0.0 had a significant performance regression (I'll take > the > > blame for not thinking through all the use cases), most easily exposed > when > > writing datasets from pandas / numpy data, which is being addressed in > > [1]. I believe this to be a fairly common use case and it may warrant a > > 12.0.1 patch. Are there other issues that would need a patch? Do we > feel > > this issue is significant enough to justify the work? > > > > [1] https://github.com/apache/arrow/pull/35565 >
