[GitHub] jihoonson commented on issue #6125: Fix appenderator_realtime creating shards bigger by 1 than maxRowsPerSegment

jihoonson commented on issue #6125: Fix appenderator_realtime creating shards bigger by 1 than maxRowsPerSegment URL: https://github.com/apache/incubator-druid/pull/6125#issuecomment-412252272 @KenjiTakahashi no, it looks not. I restarted it and now all tests have passed. I'm merging this

[GitHub] fjy closed issue #6149: Further optimize memory for Travis

fjy closed issue #6149: Further optimize memory for Travis URL: https://github.com/apache/incubator-druid/issues/6149 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] fjy commented on issue #6150: Further optimize memory for Travis jobs

fjy commented on issue #6150: Further optimize memory for Travis jobs URL: https://github.com/apache/incubator-druid/pull/6150#issuecomment-412251227  This is an automated message from the Apache Git Service. To respond to

[GitHub] fjy commented on issue #6155: Fix three bugs with segment publishing.

fjy commented on issue #6155: Fix three bugs with segment publishing. URL: https://github.com/apache/incubator-druid/pull/6155#issuecomment-412251201  This is an automated message from the Apache Git Service. To respond to

[GitHub] fjy commented on issue #6156: New doc fixes

fjy commented on issue #6156: New doc fixes URL: https://github.com/apache/incubator-druid/pull/6156#issuecomment-412251191  This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory`

asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory` URL: https://github.com/apache/incubator-druid/pull/6090#issuecomment-412243943 > they should be killed so that we can figure out what happened

[GitHub] jon-wei opened a new pull request #6156: New doc fixes

jon-wei opened a new pull request #6156: New doc fixes URL: https://github.com/apache/incubator-druid/pull/6156 Fixes some broken links and tags code blocks with `json` and `bash` where appropriate in the new tutorial/quickstart/api/configuration docs

[GitHub] gianm opened a new pull request #6155: Fix three bugs with segment publishing.

gianm opened a new pull request #6155: Fix three bugs with segment publishing. URL: https://github.com/apache/incubator-druid/pull/6155 1. In AppenderatorImpl: always use a unique path if requested, even if the segment was already pushed. This is important because if we don't do

[GitHub] pdeva commented on issue #5340: Exceptions in broker: Channel disconnected

pdeva commented on issue #5340: Exceptions in broker: Channel disconnected URL: https://github.com/apache/incubator-druid/issues/5340#issuecomment-412241334 it seems this issue is still present in 0.12.1 #6117 This is an

Build failed in Jenkins: Druid-Master-Nightly #3

See Changes: [jihoonson] Fix missing exception handling as part of -- [...truncated 419.45 KB...] at

[GitHub] gianm opened a new issue #6154: Ability to specify list of task ports

gianm opened a new issue #6154: Ability to specify list of task ports URL: https://github.com/apache/incubator-druid/issues/6154 In some environments (like containerized environments without isolated network stacks) it is desirable to decide ahead of time exactly what ports your services

[incubator-druid] branch master updated: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory` (#6090)

This is an automated email from the ASF dual-hosted git repository. jihoonson 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 bd95b42 Fix missing exception

[GitHub] jihoonson closed pull request #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory`

jihoonson closed pull request #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory` URL: https://github.com/apache/incubator-druid/pull/6090 This is a PR merged from a forked repository. As GitHub hides the original diff on

[GitHub] jihoonson opened a new pull request #6153: Fix race in testCheckpointForUnknownTaskGroup() of KafkaSupervisorTest

jihoonson opened a new pull request #6153: Fix race in testCheckpointForUnknownTaskGroup() of KafkaSupervisorTest URL: https://github.com/apache/incubator-druid/pull/6153 Fixes #6139. This is an automated message from the

Build failed in Jenkins: Druid-Master-Nightly #2

See Changes: [jon-wei] Fix redirects (#6151) -- [...truncated 2.32 MB...] at

[GitHub] jon-wei closed pull request #6151: Fix redirect to configuration doc

jon-wei closed pull request #6151: Fix redirect to configuration doc URL: https://github.com/apache/incubator-druid/pull/6151 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[incubator-druid] branch master updated: Fix redirects (#6151)

This is an automated email from the ASF dual-hosted git repository. jonwei 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 064c22c Fix redirects (#6151) 064c22c

[GitHub] a2l007 opened a new pull request #6151: Fix redirect to configuration doc

a2l007 opened a new pull request #6151: Fix redirect to configuration doc URL: https://github.com/apache/incubator-druid/pull/6151 Redirects to the newly created configuration page doesn't seem to be working properly. For instance accessing

[GitHub] jihoonson opened a new pull request #6150: Further optimize memory for Travis jobs

jihoonson opened a new pull request #6150: Further optimize memory for Travis jobs URL: https://github.com/apache/incubator-druid/pull/6150 Fixes https://github.com/apache/incubator-druid/issues/6149. I tested this PR by triggering several Travis jobs, and could see `processing`

Build failed in Jenkins: Druid-Master-Nightly #1

See -- [...truncated 2.61 MB...] [WARNING] - io.netty.util.internal.shaded.org.jctools.queues.CircularArrayOffsetCalculator [WARNING] -

[GitHub] leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r209358250 ## File path:

[GitHub] leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r209356695 ## File path:

svn commit: r28662 - in /dev/druid: ./ nightly/

Author: jihoonson Date: Fri Aug 10 18:36:25 2018 New Revision: 28662 Log: Add nightly dir Added: dev/druid/ dev/druid/nightly/ - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands,

[GitHub] jihoonson commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory`

jihoonson commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory` URL: https://github.com/apache/incubator-druid/pull/6090#issuecomment-412134432 > BTW, the No output has been received in the last 10m0s situation

[GitHub] jihoonson commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory`

jihoonson commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory` URL: https://github.com/apache/incubator-druid/pull/6090#issuecomment-412133483 @asdf2014 thanks for the patience. I've just restarted them.

[GitHub] jihoonson opened a new issue #6149: Further optimize memory for Travis

jihoonson opened a new issue #6149: Further optimize memory for Travis URL: https://github.com/apache/incubator-druid/issues/6149 Partially fixes https://github.com/apache/incubator-druid/issues/6013. Our Travis CI is frequently failing and one of the common reasons is the oom

[GitHub] drcrallen commented on issue #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

drcrallen commented on issue #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#issuecomment-412131658 Test failures: ``` Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time

[GitHub] drcrallen commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

drcrallen commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r209310422 ## File path:

[GitHub] RestfulBlue commented on issue #5006: No protections for select query

RestfulBlue commented on issue #5006: No protections for select query URL: https://github.com/apache/incubator-druid/issues/5006#issuecomment-412073339 @asdf2014 jdbc its just example, in some places native queryes preferred, as sql has some problems.i think every query should have

[GitHub] asdf2014 commented on issue #5006: No protections for select query

asdf2014 commented on issue #5006: No protections for select query URL: https://github.com/apache/incubator-druid/issues/5006#issuecomment-412035950 Hi, @RestfulBlue . Indeed, we should analyze the query sql based on cost, then we can add some limitations for malicious query to protect

[GitHub] RestfulBlue commented on issue #5006: No protections for select query

RestfulBlue commented on issue #5006: No protections for select query URL: https://github.com/apache/incubator-druid/issues/5006#issuecomment-412031674 @asdf2014 we use grafana as main gui, we have our self developed druid datasource and we have good restrictions as well. Using grafana

[GitHub] asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory`

asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory` URL: https://github.com/apache/incubator-druid/pull/6090#issuecomment-412017812 @jihoonson Thanks a lot, I have seen that travis job has been

[GitHub] asdf2014 commented on issue #5849: IntelliJ Annotation processor prevents running tests in IntelliJ

asdf2014 commented on issue #5849: IntelliJ Annotation processor prevents running tests in IntelliJ URL: https://github.com/apache/incubator-druid/issues/5849#issuecomment-412013779 I also confirmed that after upgrading Intellij Idea to v2018.2, it no longer happens. Thanks all.

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

asdf2014 removed a comment on issue #5955: The annotation handler 'io.druid.annotations.SubclassesMustBePublicAnnotationProcessor' could not be found URL: https://github.com/apache/incubator-druid/issues/5955#issuecomment-412013410 I confirm that after upgrading Intellij's idea to

[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-412013410 I confirm that after upgrading Intellij's idea to v2018.2, it no

[GitHub] asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory`

asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory` URL: https://github.com/apache/incubator-druid/pull/6090#issuecomment-411999244 Hi, @jihoonson . It seems that the failure of travis build job is