[GitHub] gianm commented on issue #5976: Update license headers.

gianm commented on issue #5976: Update license headers. URL: https://github.com/apache/incubator-druid/pull/5976#issuecomment-403188392 This is the earliest commit I can find for both of those files with no license headers. They both came from another repo that was copied into this one as

[GitHub] gianm commented on issue #5976: Update license headers.

gianm commented on issue #5976: Update license headers. URL: https://github.com/apache/incubator-druid/pull/5976#issuecomment-403188239 Most of this diff is updating one set of boilerplate headers to another. Other than that, there are a few files in the extendset package that I want to

[GitHub] gianm opened a new pull request #5976: Update license headers.

gianm opened a new pull request #5976: Update license headers. URL: https://github.com/apache/incubator-druid/pull/5976 For compliance with http://www.apache.org/legal/src-headers.html. This is an automated message from the

Re: Druid repo migration plan

I asked in our original Infra ticket to adjust the mailing lists: https://issues.apache.org/jira/browse/INFRA-16674?focusedCommentId=16535589=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16535589 Unfortunately setting up filters proved challenging in Gmail (gitbox

[GitHub] asdf2014 commented on issue #5955: The annotation handler 'io.druid.annotations.SubclassesMustBePublicAnnotationProcessor' could not be found

asdf2014 commented on issue #5955: The annotation handler 'io.druid.annotations.SubclassesMustBePublicAnnotationProcessor' could not be found URL: https://github.com/apache/incubator-druid/issues/5955#issuecomment-403180664 I see. Thanks.

[GitHub] asdf2014 closed issue #5955: The annotation handler 'io.druid.annotations.SubclassesMustBePublicAnnotationProcessor' could not be found

asdf2014 closed issue #5955: The annotation handler 'io.druid.annotations.SubclassesMustBePublicAnnotationProcessor' could not be found URL: https://github.com/apache/incubator-druid/issues/5955 This is an automated

[GitHub] jihoonson opened a new pull request #5974: [Backport] Coordinator fix balance to try to move max segments instead of up to max segments

jihoonson opened a new pull request #5974: [Backport] Coordinator fix balance to try to move max segments instead of up to max segments URL: https://github.com/apache/incubator-druid/pull/5974 Backport of #5927 to 0.12.2.

[GitHub] jihoonson opened a new pull request #5972: [Backport] Prevent KafkaSupervisor NPE in generateSequenceName

jihoonson opened a new pull request #5972: [Backport] Prevent KafkaSupervisor NPE in generateSequenceName URL: https://github.com/apache/incubator-druid/pull/5972 Backport of #5902 to 0.12.2. This is an automated message

[GitHub] jihoonson opened a new pull request #5969: [Backport] Fix inefficient available segment cache population in SQLMetadataSegmentManager

jihoonson opened a new pull request #5969: [Backport] Fix inefficient available segment cache population in SQLMetadataSegmentManager URL: https://github.com/apache/incubator-druid/pull/5969 Backport of #5878 to 0.12.2. This

[GitHub] jihoonson opened a new pull request #5968: [Backport] HdfsDataSegmentPusher: Close tmpIndexFile before copying it.

jihoonson opened a new pull request #5968: [Backport] HdfsDataSegmentPusher: Close tmpIndexFile before copying it. URL: https://github.com/apache/incubator-druid/pull/5968 Backport of #5873 to 0.12.2. This is an automated

[GitHub] jihoonson opened a new pull request #5967: [Backport] Fix mismatch in revoked task locks between memory and metastore after sync from storage

jihoonson opened a new pull request #5967: [Backport] Fix mismatch in revoked task locks between memory and metastore after sync from storage URL: https://github.com/apache/incubator-druid/pull/5967 Backport of #5858 to 0.12.2.

[GitHub] jihoonson opened a new pull request #5965: [Backport] Fix topN lexicographic sort

jihoonson opened a new pull request #5965: [Backport] Fix topN lexicographic sort URL: https://github.com/apache/incubator-druid/pull/5965 Backport of #5815 to 0.12.2. This is an automated message from the Apache Git

[incubator-druid] branch master updated: Increase timeout for BlockingPoolTest (#5959)

This is an automated email from the ASF dual-hosted git repository. gian pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new d1d9358 Increase timeout for

[GitHub] jihoonson opened a new pull request #5961: [Backport] revert lambda conversion to fix occasional jvm error

jihoonson opened a new pull request #5961: [Backport] revert lambda conversion to fix occasional jvm error URL: https://github.com/apache/incubator-druid/pull/5961 Backport of https://github.com/apache/incubator-druid/pull/5591 to 0.12.2.

[GitHub] jihoonson opened a new pull request #5959: Increase timeout for BlockingPoolTest

jihoonson opened a new pull request #5959: Increase timeout for BlockingPoolTest URL: https://github.com/apache/incubator-druid/pull/5959 Hope this fixes this Travis failure (https://github.com/apache/incubator-druid/pull/5931#issuecomment-403133666). /cc @gianm @surekhasaharan

[GitHub] jihoonson commented on issue #5941: [Backport] Fix missing task type in task payload API

jihoonson commented on issue #5941: [Backport] Fix missing task type in task payload API URL: https://github.com/apache/incubator-druid/pull/5941#issuecomment-403153636 @gianm it looks that the above failures come from another PR? The failed task in this PR was the below one, and I've

[GitHub] surekhasaharan commented on issue #5931: Support limit for timeseries query (#5894)

surekhasaharan commented on issue #5931: Support limit for timeseries query (#5894) URL: https://github.com/apache/incubator-druid/pull/5931#issuecomment-403140174 @gianm thanks for checking the TeamCity error. This is an

Re: Druid repo migration plan

IMO, GitHub has good tools already for subscribing to repos, notifications, and issues. I would expect Druid contributors to use those tools and _not_ to join git...@druid.apache.org. I'd imagine the list would only exist for archival purposes (we want a copy of every discussion to be stored

Re: Druid repo migration plan

I see there are notifications not just for commits but for every comment on every issue. That’s going to be overwhelming. I can’t imagine anyone wanting to subscribe to the gitbox list at the current volume. How about only sending comments to people who have watched a particular case? Julian

[incubator-druid] annotated tag druid-0.12.1-iap6 deleted (was db761be)

This is an automated email from the ASF dual-hosted git repository. gian pushed a change to annotated tag druid-0.12.1-iap6 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. *** WARNING: tag druid-0.12.1-iap6 was deleted! *** tag was db761be The revisions that were on

Re: Druid repo migration plan

I like the idea of a separate list for commits. Keeps the noise down on the dev list. Other projects (e.g. calcite) would name that list “commits”, i.e. comm...@druid.apache.org . If you were to add other repos (e.g. a subversion repo for web site) you could

[GitHub] gianm commented on issue #5931: Support limit for timeseries query (#5894)

gianm commented on issue #5931: Support limit for timeseries query (#5894) URL: https://github.com/apache/incubator-druid/pull/5931#issuecomment-403134175 @surekhasaharan The TeamCity error looks legit. It's complaining that this method isn't used anymore: ``` public static

[GitHub] gianm commented on issue #5931: Support limit for timeseries query (#5894)

gianm commented on issue #5931: Support limit for timeseries query (#5894) URL: https://github.com/apache/incubator-druid/pull/5931#issuecomment-403133666 Saw this in travis, restarting. ```Failed tests: BlockingPoolTest.testConcurrentBatchClose:266 expected:<5> but was:<0>

Re: Druid repo migration plan

+1 to sending to git...@druid.apache.org On Fri, Jul 6, 2018 at 12:17 PM, Gian Merlino wrote: > The repo move has happened: https://github.com/apache/incubator-druid > > My understanding is that now we can and should start prepping license, > notice, etc for a release. And also making sure that

[incubator-druid] branch 0.12.1-iap updated: [maven-release-plugin] prepare for next development iteration

This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a commit to branch 0.12.1-iap in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/0.12.1-iap by this push: new 203d5d0

[incubator-druid] annotated tag druid-0.12.1-iap6 updated (ec62856 -> db761be)

This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a change to annotated tag druid-0.12.1-iap6 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. *** WARNING: tag druid-0.12.1-iap6 was modified! *** from ec62856 (commit) to

[incubator-druid] branch 0.12.1-iap created (now ec62856)

This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a change to branch 0.12.1-iap in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. at ec62856 [maven-release-plugin] prepare release druid-0.12.1-iap6 No new revisions were added by

[GitHub] surekhasaharan opened a new pull request #5931: Support limit for timeseries query (#5894)

surekhasaharan opened a new pull request #5931: Support limit for timeseries query (#5894) URL: https://github.com/apache/incubator-druid/pull/5931 Added limit param to `TimeseriesQuery` and support to limit results in `TimeseriesQueryQueryToolChest.mergeResults` and

[GitHub] nishantmonu51 commented on issue #5958: Part 2 of changes for SQL Compatible Null Handling

nishantmonu51 commented on issue #5958: Part 2 of changes for SQL Compatible Null Handling URL: https://github.com/apache/incubator-druid/pull/5958#issuecomment-403121757 @leventov : Created a new PR, please review. This is

[GitHub] nishantmonu51 opened a new pull request #5958: Part 2 of changes for SQL Compatible Null Handling

nishantmonu51 opened a new pull request #5958: Part 2 of changes for SQL Compatible Null Handling URL: https://github.com/apache/incubator-druid/pull/5958 Part 2 of changes for SQL Compatible Null Handling - This PR contains changes to improve handling of null values in druid by

[GitHub] MTShannon commented on issue #2788: Jackson 2.4.6 incompatible with HDP 2.4

MTShannon commented on issue #2788: Jackson 2.4.6 incompatible with HDP 2.4 URL: https://github.com/apache/incubator-druid/issues/2788#issuecomment-403099742 @santhoshtangudu How did you stop passing multiple versions? This