[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-12 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285764802 ## core/src/main/java/org/apache/calcite/plan/RelOptForeignKey.java: ## @@ -0,0 +1,411 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285783800 ## core/src/test/java/org/apache/calcite/test/RelMetadataTest.java: ## @@ -971,6 +989,1025 @@ final RelMetadataFixture sql(String sql) { is(1D)); }

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285777370 ## core/src/main/java/org/apache/calcite/plan/RelOptForeignKey.java: ## @@ -0,0 +1,411 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285776926 ## core/src/main/java/org/apache/calcite/plan/RelOptForeignKey.java: ## @@ -0,0 +1,411 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285772183 ## testkit/src/main/java/org/apache/calcite/test/RelMetadataFixture.java: ## @@ -264,6 +265,18 @@ public RelMetadataFixture assertPercentageOriginalRows(Matcher

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285772183 ## testkit/src/main/java/org/apache/calcite/test/RelMetadataFixture.java: ## @@ -264,6 +265,18 @@ public RelMetadataFixture assertPercentageOriginalRows(Matcher

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285767798 ## core/src/main/java/org/apache/calcite/plan/RelOptForeignKey.java: ## @@ -0,0 +1,411 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285766949 ## core/src/main/java/org/apache/calcite/plan/RelOptForeignKey.java: ## @@ -0,0 +1,411 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285764802 ## core/src/main/java/org/apache/calcite/plan/RelOptForeignKey.java: ## @@ -0,0 +1,411 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285763589 ## core/src/main/java/org/apache/calcite/rel/metadata/BuiltInMetadata.java: ## @@ -105,6 +106,43 @@ interface Handler extends MetadataHandler { } } + /** +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285760940 ## core/src/main/java/org/apache/calcite/plan/RelOptForeignKey.java: ## @@ -0,0 +1,411 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285760940 ## core/src/main/java/org/apache/calcite/plan/RelOptForeignKey.java: ## @@ -0,0 +1,411 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285755688 ## core/src/main/java/org/apache/calcite/rel/metadata/BuiltInMetadata.java: ## @@ -105,6 +106,43 @@ interface Handler extends MetadataHandler { } } + /** +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285753858 ## core/src/main/java/org/apache/calcite/rel/metadata/BuiltInMetadata.java: ## @@ -105,6 +106,43 @@ interface Handler extends MetadataHandler { } } + /** +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285753858 ## core/src/main/java/org/apache/calcite/rel/metadata/BuiltInMetadata.java: ## @@ -105,6 +106,43 @@ interface Handler extends MetadataHandler { } } + /** +

[GitHub] [calcite] LakeShen commented on a diff in pull request #3345: [CALCITE-5881] Support to get foreign keys metadata in RelMetadataQuery

2023-08-07 Thread via GitHub
LakeShen commented on code in PR #3345: URL: https://github.com/apache/calcite/pull/3345#discussion_r1285749110 ## core/src/main/java/org/apache/calcite/plan/RelOptForeignKey.java: ## @@ -0,0 +1,411 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +