docs: Update Analyse.rst with support for Iceberg connector#27039
docs: Update Analyse.rst with support for Iceberg connector#27039Ishaan-2312 wants to merge 2 commits intoprestodb:masterfrom
Conversation
|
|
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates the SQL ANALYZE command documentation to describe support and behavior for the Iceberg connector in analyze.rst. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
steveburnett
left a comment
There was a problem hiding this comment.
Take a look at the comments and let me know if you have questions.
| @@ -22,6 +22,7 @@ connector-specific properties. To list all available properties, run the followi | |||
|
|
|||
| Currently, this statement is only supported by the | |||
| :ref:`Hive connector <hive_analyze>`. | |||
There was a problem hiding this comment.
| :ref:`Hive connector <hive_analyze>`. | |
| :ref:`Hive connector <hive_analyze>` and the :ref:`Iceberg connector <iceberg_analyze>`. |
Syntax of the Iceberg link fixed - you had added a space after the ref: in the Iceberg link so the current file looked like this in a local doc build.

Local doc builds are helpful to find formatting mistakes. For more information on running local doc builds, see Building the Documentation in the Presto docs README.
|
|
||
| Currently, this statement is only supported by the | ||
| :ref:`Hive connector <hive_analyze>`. | ||
| :ref: `Iceberg connector <iceberg_analyze>` |
in the PR description to pass the failing but not required for merge test.
Maybe something like
|
steveburnett
left a comment
There was a problem hiding this comment.
There are no changes to any files in this PR now so there is nothing to review. A valid link to the Iceberg connector was added to the ANALYZE doc in #27041, which has been merged and is already in the master branch, so a local doc build of this PR is identical to a local doc build of the master branch.
Suggest closing this PR as superseded by #27041.
When you have time, please open a new docs PR to add an example to the Iceberg doc as mentioned in this comment on 27041. I opened issue #27065 which the new docs PR would fix.

Description
docs: clarify ANALYZE connector support
This change updates documentation to reflect that the ANALYZE statement
is supported by both Hive and Iceberg connectors.
Motivation and Context
Impact
Test Plan
Contributor checklist
Release Notes
Summary by Sourcery
Documentation: