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

2023-08-14 Thread via GitHub
LakeShen commented on PR #3345: URL: https://github.com/apache/calcite/pull/3345#issuecomment-1676745035 IMO,a foreign key is one of the table constraints, the current in Calcite, we could use Statistic# getReferentialConstraints method to get the table's foreign key. So in an SQL

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

2023-08-07 Thread via GitHub
LakeShen commented on PR #3345: URL: https://github.com/apache/calcite/pull/3345#issuecomment-1668059287 Another point,Do you need add a class GeneratedMetadata_ForeignXXX like GeneratedMetadata_CollationHandler.java?

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

2023-08-01 Thread via GitHub
LakeShen commented on PR #3345: URL: https://github.com/apache/calcite/pull/3345#issuecomment-1660181104 Hi @JingDas ,your pr's workflow has some abnormal information,please check it. You could run this cmd in your local idea: ```sql ./gradlew check ``` -- This is an