prestodb / presto

The official home of the Presto distributed SQL query engine for big data
http://prestodb.io
Apache License 2.0
15.97k stars 5.35k forks source link

[docs] Combine the descriptions of session property with configuration property for history based optimization #23041

Closed hantangwangd closed 2 months ago

hantangwangd commented 3 months ago

Currently the description of session properties in presto-docs/src/main/sphinx/optimizer/history-based-optimization.rst do not combine with the corresponding configure properties. This is not in line with the actual implementation. For example, the default value of the session property should be the value of its corresponding configuration property.

A reference document for combining configuration property and session property is as follows:

https://github.com/prestodb/presto/blob/master/presto-docs/src/main/sphinx/connector/iceberg.rst?plain=1#L206-L289 https://github.com/prestodb/presto/blob/master/presto-docs/src/main/sphinx/connector/iceberg.rst?plain=1#L348-L363

Expected Behavior or Use Case

The descriptions for session property should combine with the corresponding config property in presto-docs/src/main/sphinx/optimizer/history-based-optimization.rst

Presto Component, Service, or Connector

Optimizer

Possible Implementation

N/A

Example Screenshots (if appropriate):

N/A

Context

https://github.com/prestodb/presto/blob/master/presto-docs/src/main/sphinx/connector/iceberg.rst?plain=1#L206-L289 https://github.com/prestodb/presto/blob/master/presto-docs/src/main/sphinx/connector/iceberg.rst?plain=1#L348-L363