asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitry Lychagin (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: WIP: stddev
Date Tue, 09 Oct 2018 17:40:42 GMT
Dmitry Lychagin has posted comments on this change.

Change subject: WIP: stddev
......................................................................


Patch Set 4:

One more note. We need to plan on adding support for distinct aggregate for this function.
This could be done in a follow up change. Please take a look at BuiltinFunctions.addDistinctAgg().
There's no need for runtime for those, only new function identifier definitions.

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2990
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia709669a9d20358f11ad28f453ae8ad8551f6334
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: James Fang <jfang003@ucr.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Dmitry Lychagin <dmitry.lychagin@couchbase.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Mime
View raw message