Apply suggestions from code review

Co-authored-by: Steve Burnett <[email protected]>

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

@@ -21,7 +21,7 @@ _______________
 * Add a flag to the Presto CLI which allows skipping SSL certificate verification. :pr:`23780`
 * Add a session property ``native_max_extended_partial_aggregation_memory`` which specifies Presto native max partial aggregation memory when data reduction is optimal. :pr:`23527`
 * Add a session property ``native_max_partial_aggregation_memory`` which specifies Presto native max partial aggregation memory when data reduction is not optimal. :pr:`23527`
-* Add a session property ``native_max_spill_bytes `` which specifies Presto native max allowed spill bytes. :pr:`23527`
+* Add a session property ``native_max_spill_bytes`` which specifies Presto native max allowed spill bytes. :pr:`23527`
 * Add function :func:`is_private_ip` that returns true when the input IP address is private or a reserved IP address. :pr:`23520`
 * Add function :func:`ip_prefix_subnets` that splits the input prefix into subnets the size of the new prefix length. :pr:`23656`
 * Add new configuration property ``eager-plan-validation-enabled`` for eager building of validation of a logical plan before queuing. :pr:`23649`
@@ -34,7 +34,7 @@ _______________
 * Replace configuration property ``async-cache-full-persistence-interval`` with ``async-cache-persistence-interval``. :pr:`23626`
 * Remove ``array_dupes`` and ``array_has_dupes`` alias names from functions :func:`array_duplicates` and :func:`array_has_duplicates`. :pr:`23762`
 * Improve ``element_at`` by avoiding pushdown of negative position for ``element_at`` for array.  :pr:`23479`
-* Improve GET `/v1/info/state` to return INACTIVE state until the resource group configuration manager is fully initialized. :pr:`23585`
+* Improve ``GET /v1/info/state`` to return INACTIVE state until the resource group configuration manager is fully initialized. :pr:`23585`
 * Improve IcebergDistributedSmokeTestBase by making it abstract. :pr:`23580`
 * Upgrade avro to version 1.11.4. :pr:`23868`
 * Upgrade commons-codec to version 1.17.0. :pr:`23868`
@@ -51,26 +51,26 @@ _______________
 Presto C++ (Native Execution) Changes
 ______________________________________
 * Add test for logical type storage in Parquet files. :pr:`23388`
-* Fix `task.writer-count` and `task.partitioned-writer-count` configuration properties in Presto C++ for consistency with Presto. :pr:`23902`
-* Add a metric `presto_cpp.memory_pushback_expected_reduction_bytes` to track expected reduction in memory after a pushback attempt. :pr:`23872`
+* Fix ``task.writer-count`` and ``task.partitioned-writer-count`` configuration properties in Presto C++ for consistency with Presto. :pr:`23902`
+* Add a metric ``presto_cpp.memory_pushback_expected_reduction_bytes`` to track expected reduction in memory after a pushback attempt. :pr:`23872`
 * Add a new counter, presto_cpp.memory_pushback_reduction_bytes, to monitor the actual memory reduction achieved with each memory pushback attempt. :pr:`23813`
 * Allow setting chunk size in ChunkedSliceOutput, it can reduce memory in the cases of large number of small output buffer. :pr:`23707`
-* Add ``native_max_local_exchange_partition_count`` session property which maps to the max_local_exchange_partition_count velox query property introduced in facebookincubator/velox#11292 :pr:`23910`
-* Add session property: 'native_writer_flush_threshold_bytes' which specifies the minimum memory footprint size required to reclaim memory from a file writer by flushing its buffered data to disk. :pr:`23891 `
-* Remove session property ``native_join_spiller_partition_bits``. :pr:`23906 `
-* Add session property: 'native_max_page_partitioning_buffer_size' which specifies the maximum bytes to buffer per PartitionedOutput operator to avoid creating tiny SerializedPages. :pr:`23853`
-* Add session property: 'native_max_output_buffer_size' which specifies the maximum size in bytes for the task's buffered output. The buffer is shared among all drivers. :pr:`23853`
-* Migrated arbitrator configs to use the new string-based format. :pr:`23496 `
-* Merged FilterNode into TableScanNode where possible, reducing I/O and CPU; added $path and $bucket to split info, and fixed split counts in coordinator UI. `:pr:23755`
-* Fix a bug where users weren't able to set `native_expression.max_array_size_in_reduce` session property. :pr:`23856`.
+* Add ``native_max_local_exchange_partition_count`` session property which maps to the ``max_local_exchange_partition_count`` velox query property introduced in facebookincubator/velox#11292 :pr:`23910`
+* Add session property: ``native_writer_flush_threshold_bytes`` which specifies the minimum memory footprint size required to reclaim memory from a file writer by flushing its buffered data to disk. :pr:`23891`
+* Remove session property ``native_join_spiller_partition_bits``. :pr:`23906`
+* Add session property: ``native_max_page_partitioning_buffer_size`` which specifies the maximum bytes to buffer per PartitionedOutput operator to avoid creating tiny SerializedPages. :pr:`23853`
+* Add session property: ``native_max_output_buffer_size`` which specifies the maximum size in bytes for the task's buffered output. The buffer is shared among all drivers. :pr:`23853`
+* Migrated arbitrator configs to use the new string-based format. :pr:`23496`
+* Improve I/O and CPU usage by merging FilterNode into TableScanNode where possible. Also, added ``$path`` and ``$bucket`` to split info, and fixed the split counts in the coordinator UI. :pr:`23755`
+* Fix a bug where users weren't able to set the ``native_expression.max_array_size_in_reduce`` session property. :pr:`23856`
 * Add incremental periodic cache persistence for Presto C++ worker. :pr:`23626`
 * Fix plan validation failures for some join queries running with spill enabled when using Presto C++. :pr:`23595`
 * Add native system session property provider. :pr:`23045`
 
 Security Changes
 ________________
 * Upgrade Postgres JDBC Driver to 42.6.1. :pr:`23710`
-* Upgrade the logback-core version to 1.2.13 :pr:`23735`
+* Upgrade the logback-core version to 1.2.13. :pr:`23735`
 
 Hive Connector Changes
 ______________________
@@ -90,19 +90,18 @@ _________________________
 * Add table property ``metrics_max_inferred_column`` to configure the max columns number for which metrics are collected, and support ``metrics_max_inferred_column`` for Iceberg tables with `PARQUET` format. :pr:`23468`
 * Add support for procedure ``fast_forward`` for Iceberg. :pr:`23589`
 * Support timestamp without timezone in time travel expressions :pr:`23714`.
-* Add support for using named arguments in procedure `register_table` and `unregister_table` :pr:`23592`.
-* Support new procedure set_current_snapshot for Iceberg :pr:`23567`
+* Add support for using named arguments in procedure `register_table` and `unregister_table`. :pr:`23592`.
+* Support new procedure `set_current_snapshot` for Iceberg. :pr:`23567`
 
 MongoDB Connector Changes
 _________________________
 * Add support for ``varbinary`` data type in MongoDB. :pr:`23386`
 * Add support for MongoDB ``ALTER TABLE`` statement. :pr:`23266`
-
 Cassandra Connector Changes
 ___________________________
-* Upgrade cassandra-driver-core to 3.11.5 for SSL support :pr:23493
+* Upgrade cassandra-driver-core to 3.11.5 for SSL support. :pr:`23493`
 
-Elasticsearch Changes
+Elasticsearch Connector Changes
 _____________________
 * Improve handling of exceptions for empty tables in Elasticsearch. :pr:`23850`
 
@@ -112,7 +111,7 @@ ________________
 
 Presto Parquet Changes
 ______________________
-* Fix bug so that proper logical type parameters are now read and written to Parquet files :pr:`23388`.
+* Fix bug so that proper logical type parameters are now read and written to Parquet files. :pr:`23388`
 
 SPI Changes
 ___________