Hi Eyal, I'll take a look too. For some reason I missed this when you first posted it, but it is very interesting work, and looks like it could be part of a path to supporting generic windowed aggregations in Druid SQL. (Moving average, cumulative sum, and so on)
On Tue, Jan 29, 2019 at 7:07 PM Jihoon Son <[email protected]> wrote: > Sorry for delayed review. I'll take a look once the 0.14.0 release is > finished. > > Jihoon > > On Tue, Jan 29, 2019 at 3:58 PM Eyal Yurman > <[email protected]> wrote: > > > Hi, > > > > A few months ago I worked on open sourcing an extension we've been using > > internally for a couple of years. > > > > https://github.com/apache/incubator-druid/pull/6430 > > https://github.com/apache/incubator-druid/issues/6320 > > > > This has been upvoted by a few community users, so I was happy to put in > > the effort. > > > > Unfortunately, I haven't found any committer who could spend the time > > reviewing this work... > > > > Is there anyone who could do that sometime next month? > > > > I could always release it under a separate repository, but since I am > > committed to continuing maintaining and expanding the module, I'd prefer > to > > keep it as part of the Druid codebase. > > > > Thanks! > > > > Eyal. > > >
