[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r206683837 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstAggregateCom

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r206678821 ## File path: processing/src/main/java/io/druid/query/aggregation/last/StringLastAggregatorFact

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r206679171 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstAggregatorFa

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r206331467 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstBufferAggreg

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r206319063 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstBufferAggreg

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r205902840 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstBufferAggreg

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r205900679 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstAggregateCom

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r205263804 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstAggregatorFa

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r205263503 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstAggregatorFa

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r205261555 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstAggregateCom

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r205259711 ## File path: processing/src/main/java/io/druid/query/aggregation/SerializablePairLongStringSer

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r205262817 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstAggregator.j

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r205274472 ## File path: processing/src/main/java/io/druid/query/aggregation/first/StringFirstBufferAggreg

[GitHub] jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension

jihoonson commented on a change in pull request #5789: Add stringLast and stringFirst aggregators extension URL: https://github.com/apache/incubator-druid/pull/5789#discussion_r205259258 ## File path: processing/src/main/java/io/druid/query/aggregation/SerializablePairLongStringSer