-
Notifications
You must be signed in to change notification settings - Fork 13.8k
[FLINK-28074][table-planner] show statistics details for DESCRIBE EXTENDED #20246
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update, I left some comments
...e/flink-table-common/src/main/java/org/apache/flink/table/utils/print/DescExtendedStyle.java
Outdated
Show resolved
Hide resolved
...ble/flink-table-planner/src/test/scala/org/apache/flink/table/api/TableEnvironmentTest.scala
Outdated
Show resolved
Hide resolved
3a19e86
to
5b47c7f
Compare
5b47c7f
to
adae2a1
Compare
adae2a1
to
ba75119
Compare
Thanks for your contribution, I will take a look as soon as possible. |
I think it's a very useful auxiliary command, can we move this PR forward @swuferhong @godfreyhe ? |
This PR is being marked as stale since it has not had any activity in the last 180 days. If you are having difficulty finding a reviewer, please reach out to the [community](https://flink.apache.org/what-is-flink/community/). If this PR is no longer valid or desired, please feel free to close it. If no activity occurs in the next 90 days, it will be automatically closed. |
This PR has been closed since it has not had any activity in 120 days. |
What is the purpose of the change
At first, this PR only wanted to support syntax
DESCRIBE EXTENDED
to print table statistics. But now, this PR is aims to realize syntaxDESCRIBE/DESC [EXTENDED] [catalog_name.][database_name.]table_name [PARTITION(partition_spec)] [col_name]
. Syntax implementation, get statistics from catalog and printing are mainly completed in this PRBrief change log
SqlRichDescribeTable
inparserImpls.ftl
TableEnvironmentImpl
, realizebuildDescribeResult
to build desc result.listPartitionsByFilter
method inGenericInMemoryCatalog
to support get partial partitions for genericInMemoryCatalog.desc [extended]
for none-partitioned table and partitioned table.Verifying this change
desc [extended]
for none-partitioned table and partitioned table.Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: noDocumentation
now there is no docs about this feature. will be adding