From 4c7106f30cbc92e07843123e9bb739f2375c8fe8 Mon Sep 17 00:00:00 2001 From: Jay Narale Date: Mon, 4 Nov 2024 14:10:22 -0800 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Steve Burnett --- .../src/main/sphinx/release/release-0.290.rst | 37 +++++++++---------- 1 file changed, 18 insertions(+), 19 deletions(-) diff --git a/presto-docs/src/main/sphinx/release/release-0.290.rst b/presto-docs/src/main/sphinx/release/release-0.290.rst index 36d474ef3578..ae89e1ede3a0 100644 --- a/presto-docs/src/main/sphinx/release/release-0.290.rst +++ b/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,18 +51,18 @@ _______________ 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` @@ -70,7 +70,7 @@ ______________________________________ 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 ___________