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>
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