asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Preston Carman (Code Review)" <>
Subject Change in asterixdb[master]: RangeGenerator aggfunc for the numeric/asciiString datatype ...
Date Tue, 10 May 2016 23:12:55 GMT
Preston Carman has posted comments on this change.

Change subject: RangeGenerator aggfunc for the numeric/asciiString datatype based on parallel
streaming histogram.

Patch Set 14:


I did a quick glance over the code. 

Is this only the range aggregate functions? (We still need to bake this into the compiler
to get parallel sort, correct?)
File asterixdb/asterix-app/src/test/resources/runtimets/results/aggregate/global-rg/global-avg.1.adm:

Line 1: [  ]
Why is the result empty? This seems like a bad test. Why create a range query that produces
no results?
File asterixdb/asterix-app/src/test/resources/runtimets/results/aggregate/local-rg/local-rg.1.adm:

Line 1: [  ]
Why is the result empty?
File asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/aggregates/scalar/

Line 29:     public final static FunctionIdentifier FID = AsterixBuiltinFunctions.SCALAR_RG;
Do you need this variable FID? Can this be inlined?
File asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/aggregates/std/

Line 89:                     translateVal = new BytePointable();
Use factory to create pointable.
File asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/aggregates/std/

Line 46:         // TODO Auto-generated method stub
Are these still in progress?

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I450d0962fbeacfb2b6ab9fae0750f025ef17ba01
Gerrit-PatchSet: 14
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Jianfeng Jia <>
Gerrit-Reviewer: Preston Carman <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: Yingyi Bu <>
Gerrit-Reviewer: Yingyi Bu <>
Gerrit-HasComments: Yes

View raw message