Address steve's comments round 4

Author: ZacBlanco

presto-docs/src/main/sphinx/release/release-0.292.rst

@@ -20,6 +20,9 @@ _______________
 * Improve scheduling by using long instead of DataSize for critical path. `#24582 <https://github.com/prestodb/presto/pull/24582>`_
 * Improve scheduling by using long instead of DateTime for critical path. `#24673 <https://github.com/prestodb/presto/pull/24673>`_
 * Improve presto router UI. `#24411 <https://github.com/prestodb/presto/pull/24411>`_
+* Improve how multiple operator stats are merged together. `#24414 <https://github.com/prestodb/presto/pull/24414>`_
+* Improve metrics creation by refactoring local variables to a dedicated class. `#24414 <https://github.com/prestodb/presto/pull/24414>`_
+* Improve efficiency of coordinator when running a large number of tasks, controlled by ``task.enable-event-loop``. `#24668 <https://github.com/prestodb/presto/pull/24668>`_
 * Add :doc:`../troubleshoot` topic to the Presto documentation. `#24601 <https://github.com/prestodb/presto/pull/24601>`_
 * Add Arrow Flight connector. `#24427 <https://github.com/prestodb/presto/pull/24427>`_
 * Add a MySQL-compatible function ``bit_length`` that returns the count of bits for the given string. `#24531 <https://github.com/prestodb/presto/pull/24531>`_
@@ -35,15 +38,11 @@ _______________
 * Add support for multiple query event listeners. `#24456 <https://github.com/prestodb/presto/pull/24456>`_
 * Add ``spark.dynamic-presto-memory-pool-tuning-enabled`` configuration property to dynamically configure available Spark executor memory based on available container memory. `#24714 <https://github.com/prestodb/presto/pull/24714>`_
 * Add the ability to canonicalize JSON output through session property ``canonicalized_json_extract``. `#24614 <https://github.com/prestodb/presto/pull/24614>`_
+* Add the ability for a file-based Hive metastore to use HDFS/S3 location as warehouse dir. `#24660 <https://github.com/prestodb/presto/pull/24660>`_
 * Remove org.apache.logging.log4j:log4j-api from root POM. `#24605 <https://github.com/prestodb/presto/pull/24605>`_
 * Remove org.apache.logging.log4j:log4j-core from root POM. `#24605 <https://github.com/prestodb/presto/pull/24605>`_
-* Enable file-based hive metastore to use HDFS/S3 location as warehouse dir. `#24660 <https://github.com/prestodb/presto/pull/24660>`_
-* Enable node pool type specification when reporting to the coordinator from a C++ worker. `#24569 <https://github.com/prestodb/presto/pull/24569>`_
 * Upgrade bootstrap to version 5. `#24167 <https://github.com/prestodb/presto/pull/24167>`_
 * Upgrade jQuery to version 3.7.1. `#24167 <https://github.com/prestodb/presto/pull/24167>`_
-* Improve how multiple operator stats are merged together. `#24414 <https://github.com/prestodb/presto/pull/24414>`_
-* Improve metrics creation by refactoring local variables to a dedicated class. `#24414 <https://github.com/prestodb/presto/pull/24414>`_
-* Improve efficiency of coordinator when running a large number of tasks, controlled by ``task.enable-event-loop``. `#24668 <https://github.com/prestodb/presto/pull/24668>`_
 
 Prestissimo (Native Execution) Changes
 ______________________________________
@@ -56,18 +55,22 @@ ______________________________________
     * ``shared-arbitrator.memory-pool-abort-capacity-limit``.
     * ``shared-arbitrator.memory-pool-min-reclaim-bytes``.
     * ``shared-arbitrator.memory-reclaim-threads-hw-multiplier``.
+
     `#24720 <https://github.com/prestodb/presto/pull/24720>`_
 * Add a type parameter for ``ConnectorDeleteTableHandle`` implementations to ``ConnectorProtocolTemplate``, along with support for (de)serialization of connector-specific types.  Existing native connector implementations defining ``ConnectorProtocolTemplate`` specializations must update their definitions to supply their specific type or use ``NotImplemented``. `#24721 <https://github.com/prestodb/presto/pull/24721>`_
 * Add ``exchange.http-client.request-data-sizes-max-wait-sec`` to native system configs. `#24774 <https://github.com/prestodb/presto/pull/24774>`_
 * Add ``spill-enabled``, ``join-spill-enabled``, ``aggregation-spill-enabled``, and ``order-by-spill-enabled`` to native system configs. `#24726 <https://github.com/prestodb/presto/pull/24726>`_
 * Add new error code name ``MEMORY_ARBITRATION_FAILURE`` under error code ``INSUFFICIENT_RESOURCE``. `#24773 <https://github.com/prestodb/presto/pull/24773>`_
 * Add a native function namespace manager. `#23358 <https://github.com/prestodb/presto/pull/23358>`_
 * Add support for ORC reader. `#23037 <https://github.com/prestodb/presto/pull/23037>`_
+* Add node pool type specification when reporting to the coordinator from a C++ worker. `#24569 <https://github.com/prestodb/presto/pull/24569>`_
 * Improve ``task.max-drivers-per-task`` by setting the default value to use thread concurrency of the host. `#24642 <https://github.com/prestodb/presto/pull/24642>`_
 
 Security Changes
 ________________
 * Fix a security bug when ``check_access_control_for_utlized_columns`` is true for queries that uses a ``WITH`` clause. Previously we would sometimes not check permissions for certain columns that were used in the query.  Now we will always check permissions for all columns used in the query. There are some corner cases for CTEs with the same name where we may check more columns than are used or fall back to checking all columns referenced in the query. `#24647 <https://github.com/prestodb/presto/pull/24647>`_
+* Remove reload4j dependency in response to `WS-2022-0467 <https://www.mend.io/vulnerability-database/WS-2022-0467>`_. `#24606 <https://github.com/prestodb/presto/pull/24606>`_
+* Replace deprecated ``dagre-d3`` with ``dagre-d3-es`` in response to a high severity vulnerability `WS-2022-0322 <https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2482>`_. `#24167 <https://github.com/prestodb/presto/pull/24167>`_
 * Upgrade libthrift to 0.14.1 in response to `CVE-2020-13949 <https://github.com/advisories/GHSA-g2fg-mr77-6vrm>`_. `#24462 <https://github.com/prestodb/presto/pull/24462>`_
 * Upgrade netty dependencies to version 4.1.115.Final in response to `CVE-2024-47535 <https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2024-47535>`_. `#24586 <https://github.com/prestodb/presto/pull/24586>`_
 * Upgrade prismJs to 1.30.0 in response to `CVE-2024-53382 <https://github.com/advisories/GHSA-x7hr-w5r2-h6wg>`_. `#24765 <https://github.com/prestodb/presto/pull/24765>`_
@@ -84,8 +87,6 @@ ________________
 * Upgrade json-smart version to 2.5.2 in response to `CVE-2024-57699 <https://nvd.nist.gov/vuln/detail/CVE-2024-57699>`_. `#24631 <https://github.com/prestodb/presto/pull/24631>`_
 * Upgrade the accumulo version to 1.10.1 in response to `CVE-2020-17533 <https://github.com/advisories/GHSA-grc3-8q8m-4j7c>`_. `#24438 <https://github.com/prestodb/presto/pull/24438>`_
 * Upgrade the hive-dwrf version to 0.8.7 which involved upgrading snappy version to 0.5 in response to `CVE-2024-36124 <https://github.com/advisories/GHSA-8wh2-6qhj-h7j9>`_. `#24461 <https://github.com/prestodb/presto/pull/24461>`_
-* Remove reload4j dependency in response to `WS-2022-0467 <https://www.mend.io/vulnerability-database/WS-2022-0467>`_. `#24606 <https://github.com/prestodb/presto/pull/24606>`_
-* Replace deprecated ``dagre-d3`` with ``dagre-d3-es`` in response to a high severity vulnerability `WS-2022-0322 <https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2482>`_. `#24167 <https://github.com/prestodb/presto/pull/24167>`_
 
 Elasticsearch Connector Changes
 _______________________________
@@ -130,13 +131,13 @@ ________________________
 
 SPI Changes
 ___________
+* Fix query failures by setting ``REMOTE_BUFFER_CLOSE_FAILED`` as a retriable error. `#24808 <https://github.com/prestodb/presto/pull/24808>`_
 * Add ConnectorSession as an argument to PlanChecker.validate and PlanChecker.validateFragment. `#24557 <https://github.com/prestodb/presto/pull/24557>`_
 * Add DeleteTableHandle support for the ConnectorTableHandles changes in Metadata. `#24528 <https://github.com/prestodb/presto/pull/24528>`_
 * Add ``CoordinatorPlugin#getExpressionOptimizerFactories`` to customize expression evaluation in the Presto coordinator. `#24144 <https://github.com/prestodb/presto/pull/24144>`_
 * Add a separate ConnectorDeleteTableHandle interface for ``ConnectorMetadata.beginDelete`` and ``ConnectorMetadata.finishDelete``, replacing the previous usage of ConnectorTableHandle. `#24528 <https://github.com/prestodb/presto/pull/24528>`_
+* Add IndexSourceNode to the SPI. `#24678 <https://github.com/prestodb/presto/pull/24678>`_
 * Update ``beginDelete`` to return new types, and ``finishDelete`` to accept new types in ``ConnectorMetadata``. `#24528 <https://github.com/prestodb/presto/pull/24528>`_
-* Move IndexSourceNode to the SPI. `#24678 <https://github.com/prestodb/presto/pull/24678>`_
-* Fix query failures by setting ``REMOTE_BUFFER_CLOSE_FAILED`` as a retriable error. `#24808 <https://github.com/prestodb/presto/pull/24808>`_
 
 **Credits**
 ===========