Done. Thanks for reporting the problem.

On Tuesday, April 19, 2016, Benjamin Lerer <benjamin.le...@datastax.com>
wrote:

> I will fix it. I am the one that has probably done them anyway.
>
> Benjamin
>
> On Tue, Apr 19, 2016 at 4:57 PM, Jack Krupansky <jack.krupan...@gmail.com
> <javascript:_e(%7B%7D,'cvml','jack.krupan...@gmail.com');>> wrote:
>
>> Comment typo for maxFunctionForCounter in AggregateFcts.java:
>>
>>     /**
>>      * AVG function for counter column values.
>>      */
>>     public static final AggregateFunction maxFunctionForCounter =
>>     new NativeAggregateFunction("max", CounterColumnType.instance,
>> CounterColumnType.instance)
>>
>> That comment should be "MAX" rather than "AVG"
>>
>> See:
>>
>> https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/cql3/functions/AggregateFcts.java#L760
>>
>> Oops... I see more copy and paste typos, where the type for the function
>> is
>> wrong in the comment:
>>
>> https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/cql3/functions/AggregateFcts.java#L284
>>
>> https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/cql3/functions/AggregateFcts.java#L320
>>
>> https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/cql3/functions/AggregateFcts.java#L362
>>
>> https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/cql3/functions/AggregateFcts.java#L398
>>
>> Let me know if this needs a Jia ticket or if somebody can just fix it
>> without the paperwork.
>>
>> -- Jack Krupansky
>>
>
>

Reply via email to