[GitHub] [druid] somu-imply commented on a diff in pull request #13764: Fixing a data correctness issue in unnest when first row of an MVD is null

2023-02-21 Thread via GitHub
somu-imply commented on code in PR #13764: URL: https://github.com/apache/druid/pull/13764#discussion_r1113417959 ## sql/src/main/java/org/apache/druid/sql/calcite/rel/DruidCorrelateUnnestRel.java: ## @@ -200,11 +200,36 @@ public DruidQuery toDruidQuery(boolean finalizeAggregat

[GitHub] [druid] somu-imply commented on a diff in pull request #13764: Fixing a data correctness issue in unnest when first row of an MVD is null

2023-02-08 Thread via GitHub
somu-imply commented on code in PR #13764: URL: https://github.com/apache/druid/pull/13764#discussion_r1100730560 ## sql/src/test/java/org/apache/druid/sql/calcite/util/TestDataBuilder.java: ## @@ -351,6 +351,100 @@ public Optional build( public static final List ROWS1 =