tree ee296353181f75802185e9ad2a58b0fd950bc76c
parent a993e9bf8263481b1c57546cbe58461ef6e0ab5f
author Hussain Towaileb <HussainHT@Gmail.com> 1541499160 +0300
committer Dmitry Lychagin <dmitry.lychagin@couchbase.com> 1541529683 -0800

[ASTERIXDB-2460][FUN] Fix sum() overflow bug

- user model changes: yes
- storage format changes: no
- interface changes: no

Details:
- Fixed the sum() overflow problem. Now the sum() function will
return either bigint or double based on the input types.

- Added sum() aggregate local, intermediate and global classes.
Currently, it is using the same class for the three steps and
checking the type using an if statement. Although the three
steps all have a very similar behavior, having them in split
classes makes it cleaner and easier to read and maintain in the
future.

- Added sum() aggregate local, intermediate and global for serialized
classes.

- Added sum() and serial sum() test cases and modified test cases
to cause the reported overflow issue.

- Added sum() and serial sum() test cases that cause a
Long.MAX_VALUE type overflow.

- Valid data type check is now part of the sum calculation to
avoid redundant steps.

Change-Id: I987417770b3bfbda6af29a27acc8c96dc8a99eb8
Reviewed-on: https://asterix-gerrit.ics.uci.edu/3012
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: Dmitry Lychagin <dmitry.lychagin@couchbase.com>