clintropolis opened a new pull request, #13698:
URL: https://github.com/apache/druid/pull/13698
### Description
This PR picks up where #6443 left off and merges `druid-core`,
`extendedset`, and `druid-hll` into `druid-processing`.
My hope is that this is minimally disruptive to al
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1082340975
##
processing/src/main/java/org/apache/druid/frame/file/FrameFile.java:
##
@@ -246,7 +246,8 @@ static FrameFile open(
log.warn("Could not delete frame fi
github-code-scanning[bot] commented on code in PR #13696:
URL: https://github.com/apache/druid/pull/13696#discussion_r1082214178
##
integration-tests-ex/cases/src/test/java/org/apache/druid/testsEx/query/ITJdbcQueryTest.java:
##
@@ -0,0 +1,232 @@
+/*
+ * Licensed to the Apache S
tejaswini-imply closed pull request #13681: Migrate ITs from Travis to GHA
URL: https://github.com/apache/druid/pull/13681
--
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.
To un
cryptoe commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1082148672
##
processing/src/main/java/org/apache/druid/frame/file/FrameFile.java:
##
@@ -246,7 +246,8 @@ static FrameFile open(
log.warn("Could not delete frame file
cryptoe commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1082148672
##
processing/src/main/java/org/apache/druid/frame/file/FrameFile.java:
##
@@ -246,7 +246,8 @@ static FrameFile open(
log.warn("Could not delete frame file
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081646179
##
sql/src/main/java/org/apache/druid/sql/calcite/rel/DruidCorrelateUnnestRel.java:
##
@@ -0,0 +1,300 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081630567
##
sql/src/main/java/org/apache/druid/sql/calcite/expression/Expressions.java:
##
@@ -214,12 +217,43 @@ public static DruidExpression
toDruidExpressionWithPostAggOp
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081628635
##
sql/src/main/java/org/apache/druid/sql/calcite/expression/Expressions.java:
##
@@ -214,12 +217,43 @@ public static DruidExpression
toDruidExpressionWithPostAggOp
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081627237
##
sql/src/main/java/org/apache/druid/sql/calcite/expression/Expressions.java:
##
@@ -214,12 +217,43 @@ public static DruidExpression
toDruidExpressionWithPostAggOp
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081617930
##
.idea/misc.xml:
##
@@ -84,7 +87,10 @@
http://maven.apache.org/ASSEMBLY/2.0.0";
location="$PROJECT_DIR$/.idea/xml-schemas/assembly-2.0.0.xsd" />
http://
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081616136
##
sql/src/main/java/org/apache/druid/sql/calcite/expression/Expressions.java:
##
@@ -214,12 +217,43 @@ public static DruidExpression
toDruidExpressionWithPostAggOp
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081615896
##
processing/src/test/java/org/apache/druid/segment/UnnestColumnValueSelectorCursorTest.java:
##
@@ -592,7 +590,25 @@ public void test_list_unnest_cursors_dimSelect
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081615401
##
processing/src/main/java/org/apache/druid/segment/UnnestDimensionCursor.java:
##
@@ -409,7 +412,12 @@ public int size()
@Override
public int get(int idx
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081614665
##
processing/src/main/java/org/apache/druid/segment/UnnestDimensionCursor.java:
##
@@ -181,7 +181,10 @@ public void inspectRuntimeShape(RuntimeShapeInspector
inspe
somu-imply commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1081613908
##
processing/src/main/java/org/apache/druid/query/InlineDataSource.java:
##
@@ -125,7 +125,7 @@ private static boolean rowsEqual(final Iterable
rowsA, final Iterab
paul-rogers merged PR #13625:
URL: https://github.com/apache/druid/pull/13625
--
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.
To unsubscribe, e-mail: commits-unsubscr...@druid.
rohangarg opened a new pull request, #13697:
URL: https://github.com/apache/druid/pull/13697
For large MSQ jobs with durable storage, the default number of maximum
connections (50 as of now) is sometimes a bottleneck leading to job failures.
This provides a configuration to bump the maximum
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081147729
##
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/exec/ControllerImpl.java:
##
@@ -562,13 +563,30 @@ private QueryDefinition initializeQueryDefA
cryptoe commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081143362
##
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/exec/ControllerImpl.java:
##
@@ -562,13 +563,30 @@ private QueryDefinition initializeQueryDefAnd
cryptoe commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081140648
##
processing/src/main/java/org/apache/druid/frame/processor/DurableStorageOutputChannelFactory.java:
##
@@ -0,0 +1,285 @@
+/*
+ * Licensed to the Apache Software Found
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081137315
##
.travis.yml:
##
@@ -200,7 +200,7 @@ jobs:
${MAVEN_SKIP} -Dremoteresources.skip=true
-Ddruid.generic.useDefaultValueForNull=${DRUID_USE_DEFAULT_VALUE_FO
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081136510
##
extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/output/S3StorageConnector.java:
##
@@ -89,6 +90,43 @@ public InputStream open(GetObjectRequ
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081135950
##
processing/src/main/java/org/apache/druid/frame/processor/OutputChannelFactory.java:
##
@@ -31,6 +31,11 @@
*/
OutputChannel openChannel(int partitionNumber)
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081130623
##
processing/src/main/java/org/apache/druid/frame/processor/PartitionedOutputChannel.java:
##
@@ -0,0 +1,117 @@
+/*
+ * Licensed to the Apache Software Foundation (A
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081130351
##
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/indexing/CountingOutputChannelFactory.java:
##
@@ -50,7 +51,22 @@ public OutputChannel openCha
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081128967
##
processing/src/main/java/org/apache/druid/frame/file/FrameFileFooter.java:
##
@@ -0,0 +1,126 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081128669
##
processing/src/main/java/org/apache/druid/frame/processor/DurableStorageOutputChannelFactory.java:
##
@@ -0,0 +1,285 @@
+/*
+ * Licensed to the Apache Software Fou
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081128163
##
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/exec/ControllerImpl.java:
##
@@ -562,13 +563,30 @@ private QueryDefinition initializeQueryDefA
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081125913
##
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/util/MultiStageQueryContext.java:
##
@@ -88,14 +94,30 @@ public static String getMSQMode(final
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081125379
##
processing/src/main/java/org/apache/druid/frame/channel/ByteTracker.java:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081125163
##
processing/src/main/java/org/apache/druid/frame/channel/ByteTracker.java:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081124937
##
processing/src/main/java/org/apache/druid/frame/channel/ByteTracker.java:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081124120
##
processing/src/main/java/org/apache/druid/frame/channel/ComposingReadableFrameChannel.java:
##
@@ -0,0 +1,147 @@
+/*
+ * Licensed to the Apache Software Foundation
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081123621
##
processing/src/main/java/org/apache/druid/frame/channel/ComposingReadableFrameChannel.java:
##
@@ -0,0 +1,147 @@
+/*
+ * Licensed to the Apache Software Foundation
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081123338
##
processing/src/main/java/org/apache/druid/frame/channel/ComposingReadableFrameChannel.java:
##
@@ -0,0 +1,147 @@
+/*
+ * Licensed to the Apache Software Foundation
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081122762
##
processing/src/main/java/org/apache/druid/frame/channel/ComposingReadableFrameChannel.java:
##
@@ -0,0 +1,147 @@
+/*
+ * Licensed to the Apache Software Foundation
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081122205
##
processing/src/main/java/org/apache/druid/frame/channel/ComposingWritableFrameChannel.java:
##
@@ -0,0 +1,99 @@
+/*
+ * Licensed to the Apache Software Foundation
rohangarg commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1081120486
##
processing/src/main/java/org/apache/druid/frame/channel/ComposingWritableFrameChannel.java:
##
@@ -0,0 +1,99 @@
+/*
+ * Licensed to the Apache Software Foundation
findingrish closed pull request #13661: Update exception message for ingestion
parse errors
URL: https://github.com/apache/druid/pull/13661
--
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 speci
abhagraw opened a new pull request, #13696:
URL: https://github.com/apache/druid/pull/13696
Migrated the old query group tests to the new IT framework.
This PR migrates the following tests -
ITTwitterQueryTest
ITBroadcastJoinQueryTest
ITSqlCancelTest
ITSystemTableQueryT
adarshsanjeev commented on code in PR #13674:
URL: https://github.com/apache/druid/pull/13674#discussion_r1080913156
##
sql/src/main/codegen/includes/delete.ftl:
##
@@ -0,0 +1,69 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor licens
AmatyaAvadhanula commented on issue #13692:
URL: https://github.com/apache/druid/issues/13692#issuecomment-1396563950
1. Could you also please share how long your coordinator runs take and which
duty takes the most time as well? (`coordinator/time` metric)
2. Which cost balancer strategy
kfaraz commented on PR #13683:
URL: https://github.com/apache/druid/pull/13683#issuecomment-1396539515
The lines above seem misaligned too. Maybe we align these too in this PR?
https://github.com/apache/druid/blob/fb26a1093d11f0574dbb905ac0d39c8eb3777312/docs/tutorials/tutorial-jupyte
abhishekagarwal87 commented on issue #13692:
URL: https://github.com/apache/druid/issues/13692#issuecomment-1396533209
Also tagging @kfaraz and @AmatyaAvadhanula if they have some more
suggestions.
--
This is an automated message from the Apache Git Service.
To respond to the message, pl
abhishekagarwal87 commented on issue #13692:
URL: https://github.com/apache/druid/issues/13692#issuecomment-1396532889
Wow. that's a pretty impressive scale.
I would definitely recommend upgrading to 25.0. However, that alone won't be
sufficient. There are some config changes require
abhagraw commented on code in PR #13669:
URL: https://github.com/apache/druid/pull/13669#discussion_r1080856667
##
integration-tests-ex/docs/compose.md:
##
@@ -236,3 +236,56 @@ To define a test cluster, do the following:
* If you need multiple instances of the same service, ext
kfaraz opened a new pull request, #13695:
URL: https://github.com/apache/druid/pull/13695
# Changes
- Update value of `druid.manager.lookups.period` in docs to `120_000`, which
is the value used in `LookupCoordinatorManagerConfig`
- Remove statements that are not pertinent anymore
--
abhagraw commented on code in PR #13669:
URL: https://github.com/apache/druid/pull/13669#discussion_r1080827405
##
integration-tests-ex/cases/cluster.sh:
##
@@ -137,17 +142,34 @@ function build_shared_dir {
chmod -R a+rwx $SHARED_DIR
}
-# Each test must have a default
abhagraw commented on code in PR #13669:
URL: https://github.com/apache/druid/pull/13669#discussion_r1080856667
##
integration-tests-ex/docs/compose.md:
##
@@ -236,3 +236,56 @@ To define a test cluster, do the following:
* If you need multiple instances of the same service, ext
imply-cheddar commented on code in PR #13672:
URL: https://github.com/apache/druid/pull/13672#discussion_r1080772845
##
core/src/main/java/org/apache/druid/data/input/impl/DimensionsSpec.java:
##
@@ -78,15 +80,16 @@ public static Builder builder()
public DimensionsSpec(List
cryptoe commented on code in PR #13670:
URL: https://github.com/apache/druid/pull/13670#discussion_r1080770286
##
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/indexing/error/MSQWarningReportLimiterPublisher.java:
##
@@ -108,6 +115,12 @@ public void publis
imply-cheddar commented on code in PR #13694:
URL: https://github.com/apache/druid/pull/13694#discussion_r1080766048
##
processing/src/main/java/org/apache/druid/query/rowsandcols/RearrangedRowsAndColumns.java:
##
@@ -72,13 +85,14 @@
@Override
public int numRows()
{
-
imply-cheddar commented on code in PR #13576:
URL: https://github.com/apache/druid/pull/13576#discussion_r1080745376
##
processing/src/main/java/org/apache/druid/segment/UnnestDimensionCursor.java:
##
@@ -181,7 +181,10 @@ public void inspectRuntimeShape(RuntimeShapeInspector
in
github-code-scanning[bot] commented on code in PR #13694:
URL: https://github.com/apache/druid/pull/13694#discussion_r1080744790
##
processing/src/main/java/org/apache/druid/query/operator/join/SortedInnerJoinOperator.java:
##
@@ -0,0 +1,550 @@
+/*
+ * Licensed to the Apache Sof
imply-cheddar opened a new pull request, #13694:
URL: https://github.com/apache/druid/pull/13694
This enables "merge" style operations that combine multiple streams.
This change includes a naive implementation of one such merge operator just
to provide concrete evidence that the refac
TSFenwick opened a new pull request, #13693:
URL: https://github.com/apache/druid/pull/13693
Fixes #.
### Description
Fixed the bug ...
Renamed the class ...
Added a forbidden-apis entry ...
clintropolis merged PR #13672:
URL: https://github.com/apache/druid/pull/13672
--
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.
To unsubscribe, e-mail: commits-unsubscr...@druid
maytasm merged PR #13689:
URL: https://github.com/apache/druid/pull/13689
--
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.
To unsubscribe, e-mail: commits-unsubscr...@druid.apac
benkrug commented on code in PR #13671:
URL: https://github.com/apache/druid/pull/13671#discussion_r1074028863
##
docs/querying/sql-operators.md:
##
@@ -37,6 +37,10 @@ sidebar_label: "Operators"
Operators in [Druid SQL](./sql.md) typically operate on one or two values and
re
benkrug commented on code in PR #13671:
URL: https://github.com/apache/druid/pull/13671#discussion_r1073979796
##
docs/querying/sql-operators.md:
##
@@ -37,6 +37,10 @@ sidebar_label: "Operators"
Operators in [Druid SQL](./sql.md) typically operate on one or two values and
re
paul-rogers commented on code in PR #13671:
URL: https://github.com/apache/druid/pull/13671#discussion_r1073918523
##
docs/querying/sql-operators.md:
##
@@ -37,6 +37,10 @@ sidebar_label: "Operators"
Operators in [Druid SQL](./sql.md) typically operate on one or two values and
paul-rogers commented on code in PR #13674:
URL: https://github.com/apache/druid/pull/13674#discussion_r1073897566
##
sql/src/main/codegen/includes/delete.ftl:
##
@@ -0,0 +1,69 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license
paul-rogers commented on PR #13674:
URL: https://github.com/apache/druid/pull/13674#issuecomment-1387496613
This is a change to SQL syntax. SQL syntax changes typically involve much
discussion as we want to make the syntax a SQL-compatible as possible, but we
usually need some Druid-specifi
paul-rogers commented on code in PR #13681:
URL: https://github.com/apache/druid/pull/13681#discussion_r1073851602
##
.github/workflows/reusable-revised-its.yml:
##
@@ -0,0 +1,107 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agr
acherla opened a new issue, #13692:
URL: https://github.com/apache/druid/issues/13692
Hi Druid team,
We manage a cluster that has over 3K+ peons (116 middle managers) and 200+
historical nodes (4-5 million segments) with two masters configured as both
coordinator/overlord. We run close
paul-rogers commented on code in PR #13691:
URL: https://github.com/apache/druid/pull/13691#discussion_r1073840092
##
sql/src/main/java/org/apache/druid/sql/AbstractStatement.java:
##
@@ -160,7 +160,12 @@ protected void authorize(
// Authentication is done by the planner
findingrish commented on code in PR #13682:
URL: https://github.com/apache/druid/pull/13682#discussion_r1073825727
##
core/src/main/java/org/apache/druid/math/expr/Function.java:
##
@@ -3681,4 +3681,58 @@ protected HumanReadableBytes.UnitSystem getUnitSystem()
return Huma
findingrish commented on code in PR #13682:
URL: https://github.com/apache/druid/pull/13682#discussion_r1073825727
##
core/src/main/java/org/apache/druid/math/expr/Function.java:
##
@@ -3681,4 +3681,58 @@ protected HumanReadableBytes.UnitSystem getUnitSystem()
return Huma
findingrish closed pull request #13682: `date_expand` function
URL: https://github.com/apache/druid/pull/13682
--
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.
To unsubscribe, e
somu-imply commented on PR #13682:
URL: https://github.com/apache/druid/pull/13682#issuecomment-1387409421
Please add some testcases to `FunctionTest.java`. We can add the sql
versions in the `CalciteQueryTest.java`. Additionally `sql.md` and
`website/.spelling` should include the `date_exp
paul-rogers merged PR #13684:
URL: https://github.com/apache/druid/pull/13684
--
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.
To unsubscribe, e-mail: commits-unsubscr...@druid.
adarshsanjeev opened a new pull request, #13691:
URL: https://github.com/apache/druid/pull/13691
Adds logging and for the case that an exception is thrown during
authorization
This PR has:
- [x] been self-reviewed.
- [ ] using the [concurrency
checklist](
petermarshallio opened a new pull request, #367:
URL: https://github.com/apache/druid-website-src/pull/367
Quick one - made the requirement for node 10 explicit.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
github-code-scanning[bot] commented on code in PR #13368:
URL: https://github.com/apache/druid/pull/13368#discussion_r1073524878
##
processing/src/main/java/org/apache/druid/frame/channel/ByteTracker.java:
##
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Apache Software Foundation (A
stale[bot] commented on PR #11240:
URL: https://github.com/apache/druid/pull/11240#issuecomment-1386854623
This pull request/issue is no longer marked as stale.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
abhishekagarwal87 merged PR #13690:
URL: https://github.com/apache/druid/pull/13690
--
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.
To unsubscribe, e-mail: commits-unsubscr...@
abhishekagarwal87 commented on PR #13690:
URL: https://github.com/apache/druid/pull/13690#issuecomment-1386853354
@vogievetsky FYI.
--
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 c
abhishekagarwal87 commented on PR #13617:
URL: https://github.com/apache/druid/pull/13617#issuecomment-1386728166
@toughrogrammer - can you look into CI-CD failures?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
abhishekagarwal87 merged PR #13687:
URL: https://github.com/apache/druid/pull/13687
--
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.
To unsubscribe, e-mail: commits-unsubscr...@
tejaswini-imply opened a new pull request, #13690:
URL: https://github.com/apache/druid/pull/13690
GitHub workflow `unit-tests` needs node installed. Although GitHub-provided
runners have node installed by default, some of the self-hosted runners might
not have the same installed. This PR i
maytasm commented on code in PR #13689:
URL: https://github.com/apache/druid/pull/13689#discussion_r1073175927
##
server/src/main/java/org/apache/druid/segment/indexing/ReaderUtils.java:
##
@@ -60,6 +60,12 @@ public static Set getColumnsRequiredForIngestion(
// Find colum
clintropolis commented on code in PR #13689:
URL: https://github.com/apache/druid/pull/13689#discussion_r1073145521
##
server/src/main/java/org/apache/druid/segment/indexing/ReaderUtils.java:
##
@@ -60,6 +60,12 @@ public static Set getColumnsRequiredForIngestion(
// Find
maytasm opened a new pull request, #13689:
URL: https://github.com/apache/druid/pull/13689
Fix Parquet Reader for schema-less ingestion need to read all columns
### Description
When we have either schema-less ingestion without having a flattenSpec
(which set useFieldDiscovery t
somu-imply commented on code in PR #13682:
URL: https://github.com/apache/druid/pull/13682#discussion_r1073115716
##
core/src/main/java/org/apache/druid/math/expr/Function.java:
##
@@ -3681,4 +3681,58 @@ protected HumanReadableBytes.UnitSystem getUnitSystem()
return Human
clintropolis commented on code in PR #13682:
URL: https://github.com/apache/druid/pull/13682#discussion_r1073105114
##
core/src/main/java/org/apache/druid/math/expr/Function.java:
##
@@ -3681,4 +3681,58 @@ protected HumanReadableBytes.UnitSystem getUnitSystem()
return Hum
abhishekagarwal87 commented on code in PR #13682:
URL: https://github.com/apache/druid/pull/13682#discussion_r1073103928
##
core/src/main/java/org/apache/druid/math/expr/Function.java:
##
@@ -3681,4 +3681,58 @@ protected HumanReadableBytes.UnitSystem getUnitSystem()
retur
zachjsh commented on PR #13627:
URL: https://github.com/apache/druid/pull/13627#issuecomment-1386467104
@paul-rogers , hmm it doesnt seem to give me the ability to approve the pr
anymore 😢 . But I do approve of change.
--
This is an automated message from the Apache Git Service.
T
laglangyue closed issue #13688: How to use meta API?
URL: https://github.com/apache/druid/issues/13688
--
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.
To unsubscribe, e-mail: c
laglangyue opened a new issue, #13688:
URL: https://github.com/apache/druid/issues/13688
/druid/coordinator/v1/metadata/segments?datasources=rollup-tutorial
I want to use this api by invoke it to broker,but it return 404.
how to use it ?
our clusters don't have router,so
laglangyue closed issue #13658: 0.16.1 No known server /
java.lang.RuntimeException
URL: https://github.com/apache/druid/issues/13658
--
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 com
github-code-scanning[bot] commented on code in PR #13686:
URL: https://github.com/apache/druid/pull/13686#discussion_r1073002223
##
sql/src/main/java/org/apache/druid/sql/calcite/table/DatasourceTable.java:
##
@@ -181,9 +300,10 @@
public String toString()
{
// Don't i
yurmix opened a new pull request, #13687:
URL: https://github.com/apache/druid/pull/13687
This simple docs PR fix several broken links to docs.oracle.com. These links
were broken due to missing .html (perhaps the server has stopped accepting the
original links).
--
This is an automated m
davidgmonical commented on issue #13685:
URL: https://github.com/apache/druid/issues/13685#issuecomment-1386316165
Nothing underway but happy to take a stab at it. Would love a review once I
have it up thank you!
--
This is an automated message from the Apache Git Service.
To respond to t
paul-rogers opened a new pull request, #13686:
URL: https://github.com/apache/druid/pull/13686
This PR is currently a draft.
Prior PRs added the catalog (table metadata) foundations, and an improved
set of table functions. This PR brings it all together:
* Validates the MSQ INS
benkrug commented on PR #13671:
URL: https://github.com/apache/druid/pull/13671#issuecomment-1386275064
As far as max/min bounds, this seems accurate?
https://cs.fit.edu/~ryan/java/language/java-data.html (see the "Numeric"
section.)
So, for long, it's 2^63 (since one bit is for sign
paul-rogers commented on PR #13627:
URL: https://github.com/apache/druid/pull/13627#issuecomment-1386246212
@maytasm, thanks for checking. @zachjsh had given his approval, but I see it
wasn't actually recorded here. @zachjsh, can you do the honors? Thanks!
--
This is an automated messag
clintropolis commented on code in PR #13672:
URL: https://github.com/apache/druid/pull/13672#discussion_r1072946806
##
core/src/main/java/org/apache/druid/java/util/common/parsers/JSONFlattenerMaker.java:
##
@@ -57,15 +57,21 @@ public class JSONFlattenerMaker implements
ObjectF
clintropolis commented on code in PR #13672:
URL: https://github.com/apache/druid/pull/13672#discussion_r1072898133
##
core/src/main/java/org/apache/druid/java/util/common/parsers/JSONFlattenerMaker.java:
##
@@ -57,15 +57,21 @@ public class JSONFlattenerMaker implements
ObjectF
clintropolis commented on code in PR #13682:
URL: https://github.com/apache/druid/pull/13682#discussion_r1072896301
##
core/src/main/java/org/apache/druid/math/expr/Function.java:
##
@@ -3681,4 +3681,58 @@ protected HumanReadableBytes.UnitSystem getUnitSystem()
return Hum
1 - 100 of 3064 matches
Mail list logo