[druid] branch master updated: Issue 4909 popped up again. I applied PR 5451 liberally to all new Calcite test classes introduced in PR 9279 to fix (#9324)
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/master by this push: new 5befd40 Issue 4909 popped up again. I applied PR 5451 liberally to all new Calcite test classes introduced in PR 9279 to fix (#9324) 5befd40 is described below commit 5befd406384c8717cec558fa3f7d3d86d18145f0 Author: Lucas Capistrant AuthorDate: Mon Feb 17 00:29:43 2020 -0600 Issue 4909 popped up again. I applied PR 5451 liberally to all new Calcite test classes introduced in PR 9279 to fix (#9324) --- .../org/apache/druid/sql/calcite/planner/CalcitePlannerModuleTest.java | 3 ++- .../apache/druid/sql/calcite/schema/DruidCalciteSchemaModuleTest.java | 3 ++- .../java/org/apache/druid/sql/calcite/schema/NamedDruidSchemaTest.java | 3 ++- .../org/apache/druid/sql/calcite/schema/NamedLookupSchemaTest.java | 3 ++- .../org/apache/druid/sql/calcite/schema/NamedSystemSchemaTest.java | 3 ++- .../org/apache/druid/sql/calcite/schema/RootSchemaProviderTest.java| 3 ++- 6 files changed, 12 insertions(+), 6 deletions(-) diff --git a/sql/src/test/java/org/apache/druid/sql/calcite/planner/CalcitePlannerModuleTest.java b/sql/src/test/java/org/apache/druid/sql/calcite/planner/CalcitePlannerModuleTest.java index 694f435..e081e1f 100644 --- a/sql/src/test/java/org/apache/druid/sql/calcite/planner/CalcitePlannerModuleTest.java +++ b/sql/src/test/java/org/apache/druid/sql/calcite/planner/CalcitePlannerModuleTest.java @@ -36,6 +36,7 @@ import org.apache.druid.sql.calcite.aggregation.SqlAggregator; import org.apache.druid.sql.calcite.expression.SqlOperatorConversion; import org.apache.druid.sql.calcite.schema.DruidSchemaName; import org.apache.druid.sql.calcite.schema.NamedSchema; +import org.apache.druid.sql.calcite.util.CalciteTestBase; import org.easymock.EasyMock; import org.easymock.EasyMockRunner; import org.easymock.Mock; @@ -49,7 +50,7 @@ import javax.validation.Validator; import java.util.Set; @RunWith(EasyMockRunner.class) -public class CalcitePlannerModuleTest +public class CalcitePlannerModuleTest extends CalciteTestBase { private static final String SCHEMA_1 = "SCHEMA_1"; private static final String SCHEMA_2 = "SCHEMA_2"; diff --git a/sql/src/test/java/org/apache/druid/sql/calcite/schema/DruidCalciteSchemaModuleTest.java b/sql/src/test/java/org/apache/druid/sql/calcite/schema/DruidCalciteSchemaModuleTest.java index 4f1fda0..edba60a 100644 --- a/sql/src/test/java/org/apache/druid/sql/calcite/schema/DruidCalciteSchemaModuleTest.java +++ b/sql/src/test/java/org/apache/druid/sql/calcite/schema/DruidCalciteSchemaModuleTest.java @@ -42,6 +42,7 @@ import org.apache.druid.server.QueryLifecycleFactory; import org.apache.druid.server.security.AuthorizerMapper; import org.apache.druid.server.security.Escalator; import org.apache.druid.sql.calcite.planner.PlannerConfig; +import org.apache.druid.sql.calcite.util.CalciteTestBase; import org.apache.druid.sql.calcite.view.ViewManager; import org.easymock.EasyMock; import org.easymock.EasyMockRunner; @@ -55,7 +56,7 @@ import java.util.Set; import java.util.stream.Collectors; @RunWith(EasyMockRunner.class) -public class DruidCalciteSchemaModuleTest +public class DruidCalciteSchemaModuleTest extends CalciteTestBase { private static final String DRUID_SCHEMA_NAME = "druid"; diff --git a/sql/src/test/java/org/apache/druid/sql/calcite/schema/NamedDruidSchemaTest.java b/sql/src/test/java/org/apache/druid/sql/calcite/schema/NamedDruidSchemaTest.java index eb63408..f4af22b 100644 --- a/sql/src/test/java/org/apache/druid/sql/calcite/schema/NamedDruidSchemaTest.java +++ b/sql/src/test/java/org/apache/druid/sql/calcite/schema/NamedDruidSchemaTest.java @@ -19,6 +19,7 @@ package org.apache.druid.sql.calcite.schema; +import org.apache.druid.sql.calcite.util.CalciteTestBase; import org.easymock.EasyMockRunner; import org.easymock.Mock; import org.junit.Assert; @@ -27,7 +28,7 @@ import org.junit.Test; import org.junit.runner.RunWith; @RunWith(EasyMockRunner.class) -public class NamedDruidSchemaTest +public class NamedDruidSchemaTest extends CalciteTestBase { private static final String SCHEMA_NAME = "SCHEMA_NAME"; diff --git a/sql/src/test/java/org/apache/druid/sql/calcite/schema/NamedLookupSchemaTest.java b/sql/src/test/java/org/apache/druid/sql/calcite/schema/NamedLookupSchemaTest.java index f81cbd2..22aa9e9 100644 --- a/sql/src/test/java/org/apache/druid/sql/calcite/schema/NamedLookupSchemaTest.java +++ b/sql/src/test/java/org/apache/druid/sql/calcite/schema/NamedLookupSchemaTest.java @@ -19,6 +19,7 @@ package org.apache.druid.sql.calcite.schema; +import org.apache.druid.sql.calcite.util.CalciteTestBase; import org.easymock.EasyMockRunner; import org.easymock.Mock; import org.junit.Assert; @@ -27,7 +28,7 @@ import org.junit.Test; import org.junit
[GitHub] [druid] clintropolis merged pull request #9324: Fix failing Calcite unit tests with existing work-a-round for calcite property setup
clintropolis merged pull request #9324: Fix failing Calcite unit tests with existing work-a-round for calcite property setup URL: https://github.com/apache/druid/pull/9324 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[GitHub] [druid] pktippa closed issue #9369: Doc: SqlFirehose(Postgres) to Input source.
pktippa closed issue #9369: Doc: SqlFirehose(Postgres) to Input source. URL: https://github.com/apache/druid/issues/9369 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[GitHub] [druid] pktippa commented on issue #9369: Doc: SqlFirehose(Postgres) to Input source.
pktippa commented on issue #9369: Doc: SqlFirehose(Postgres) to Input source. URL: https://github.com/apache/druid/issues/9369#issuecomment-586809109 thanks @jihoonson This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[GitHub] [druid] stale[bot] closed pull request #8698: add explodeSpec
stale[bot] closed pull request #8698: add explodeSpec URL: https://github.com/apache/druid/pull/8698 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[GitHub] [druid] stale[bot] commented on issue #8698: add explodeSpec
stale[bot] commented on issue #8698: add explodeSpec URL: https://github.com/apache/druid/pull/8698#issuecomment-586778487 This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[GitHub] [druid] jihoonson merged pull request #9353: Inject things instead of subclassing everything for parallel task testing
jihoonson merged pull request #9353: Inject things instead of subclassing everything for parallel task testing URL: https://github.com/apache/druid/pull/9353 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[druid] branch master updated (b1be88d -> 3bb9e7e)
This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from b1be88d fix Expressions.toQueryGranularity to be more correct, improve javadocs of Expr.getIdentifierIfIdentifier and Expr.getBindingIfIdentifier (#9363) add 3bb9e7e Inject things instead of subclassing everything for parallel task testing (#9353) No new revisions were added by this update. Summary of changes: .../druid/indexing/common/task/AbstractTask.java | 3 +- .../druid/indexing/common/task/CompactionTask.java | 6 + .../batch/parallel/GenericPartitionLocation.java | 9 +- .../task/batch/parallel/HashPartitionLocation.java | 7 +- .../task/batch/parallel/HttpShuffleClient.java | 87 .../InputSourceSplitParallelIndexTaskRunner.java | 6 +- ...mensionDistributionParallelIndexTaskRunner.java | 3 +- .../parallel/PartialGenericSegmentMergeTask.java | 4 +- ...HashSegmentGenerateParallelIndexTaskRunner.java | 33 +- .../parallel/PartialHashSegmentMergeTask.java | 4 +- ...angeSegmentGenerateParallelIndexTaskRunner.java | 42 +- .../batch/parallel/PartialSegmentMergeTask.java| 43 +- .../task/batch/parallel/PartitionLocation.java | 7 + ...upervisorTaskAccess.java => ShuffleClient.java} | 40 +- .../apache/druid/indexing/common/TestUtils.java| 28 -- .../common/task/CompactionTaskParallelRunTest.java | 158 +- .../common/task/CompactionTaskRunTest.java | 4 +- .../indexing/common/task/IngestionTestBase.java| 4 +- .../AbstractMultiPhaseParallelIndexingTest.java| 54 +- .../AbstractParallelIndexSupervisorTaskTest.java | 544 + ...ashPartitionMultiPhaseParallelIndexingTest.java | 197 +--- .../task/batch/parallel/HttpShuffleClientTest.java | 211 .../parallel/ParallelIndexPhaseRunnerTest.java | 33 +- .../ParallelIndexSupervisorTaskKillTest.java | 51 +- .../ParallelIndexSupervisorTaskResourceTest.java | 55 +-- .../parallel/ParallelIndexTestingFactory.java | 21 +- .../PartialGenericSegmentMergeTaskTest.java| 6 +- .../parallel/PartialHashSegmentMergeTaskTest.java | 6 +- .../PartialRangeSegmentGenerateTaskTest.java | 7 +- ...ngePartitionMultiPhaseParallelIndexingTest.java | 235 + .../parallel/SinglePhaseParallelIndexingTest.java | 206 ++-- .../realtime/firehose/LocalFirehoseFactory.java| 5 +- .../main/java/org/apache/druid/cli/CliIndexer.java | 5 - .../org/apache/druid/cli/CliMiddleManager.java | 2 + .../java/org/apache/druid/cli/CliOverlord.java | 5 +- .../main/java/org/apache/druid/cli/CliPeon.java| 14 +- 36 files changed, 967 insertions(+), 1178 deletions(-) create mode 100644 indexing-service/src/main/java/org/apache/druid/indexing/common/task/batch/parallel/HttpShuffleClient.java copy indexing-service/src/main/java/org/apache/druid/indexing/common/task/batch/parallel/{SupervisorTaskAccess.java => ShuffleClient.java} (51%) create mode 100644 indexing-service/src/test/java/org/apache/druid/indexing/common/task/batch/parallel/HttpShuffleClientTest.java - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[GitHub] [druid] stale[bot] commented on issue #8666: resolve resolveip command not found issue in integration test
stale[bot] commented on issue #8666: resolve resolveip command not found issue in integration test URL: https://github.com/apache/druid/pull/8666#issuecomment-586746465 This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the d...@druid.apache.org list. Thank you for your contributions. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[druid] branch master updated (6d52edd -> b1be88d)
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 6d52edd Remove references to Docker Machine (#9366) add b1be88d fix Expressions.toQueryGranularity to be more correct, improve javadocs of Expr.getIdentifierIfIdentifier and Expr.getBindingIfIdentifier (#9363) No new revisions were added by this update. Summary of changes: core/src/main/java/org/apache/druid/math/expr/Expr.java| 7 +-- .../java/org/apache/druid/sql/calcite/expression/Expressions.java | 2 +- 2 files changed, 6 insertions(+), 3 deletions(-) - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[GitHub] [druid] fjy merged pull request #9363: fix druid-sql Expressions.toQueryGranularity to be more correct
fjy merged pull request #9363: fix druid-sql Expressions.toQueryGranularity to be more correct URL: https://github.com/apache/druid/pull/9363 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
[GitHub] [druid] BigRantLing closed issue #9364: DirectDruidClient openConnections may be samller than the real connctions between the servers
BigRantLing closed issue #9364: DirectDruidClient openConnections may be samller than the real connctions between the servers URL: https://github.com/apache/druid/issues/9364 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org