Yes. I have the PR for this : https://github.com/apache/helix/pull/91
Best, Junkai On Wed, May 10, 2017 at 12:08 PM, kishore g <g.kish...@gmail.com> wrote: > Yes. Do you have a PR for that?. I can review it > > On Wed, May 10, 2017 at 11:19 AM, Xue Junkai <junkai....@gmail.com> wrote: > > > Sure! Please let me know if this change works or not. BTW will customized > > batch message threadpool be involved in this release? > > > > Best, > > > > Junkai > > > > On Tue, May 9, 2017 at 7:28 PM, kishore g <g.kish...@gmail.com> wrote: > > > > > I would like to have that fix included for Pinot. I will test it the > > patch. > > > > > > On Tue, May 9, 2017 at 5:59 PM, Xue Junkai <junkai....@gmail.com> > wrote: > > > > > > > It does contain the batchMessage thread pool fix. But for race > > condition > > > > fix I withdraw the pull request since I am not quite sure whether the > > fix > > > > works or not. In addition, this release will include the > > > > AutoRebalanceStrategy not assign replicas fix. > > > > > > > > > > > > Best, > > > > > > > > Junkai > > > > > > > > On Tue, May 9, 2017 at 5:49 PM, kishore g <g.kish...@gmail.com> > wrote: > > > > > > > > > Does this include the batchMessage thread pool fix and fix to the > > race > > > > > condition > > > > > > > > > > On Tue, May 9, 2017 at 5:08 PM, Xue Junkai <junkai....@gmail.com> > > > wrote: > > > > > > > > > > > Hi Helix Devs, > > > > > > > > > > > > I am going to work on releasing Helix 0.6.8 this week. Please let > > me > > > > know > > > > > > if you have any questions, comments and concerns. > > > > > > > > > > > > Best, > > > > > > > > > > > > Junkai Xue > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > Junkai Xue > > > > > > > > > > > > > > > -- > > Junkai Xue > > > -- Junkai Xue
