Copilot commented on code in PR #51657: URL: https://github.com/apache/arrow/pull/51657#discussion_r4144688624
########## r/NEWS.md: ########## @@ -19,13 +19,43 @@ # arrow 25.0.1.9000 +## Breaking changes + +- `read_feather()` and `write_feather()` now warn that they are deprecated. Use + `read_ipc_file()` and `write_ipc_file()` instead. Similarly, + `format = "feather"` in `open_dataset()` and `write_dataset()` is deprecated + in favour of `format = "ipc"`, and extra arguments passed via `...` to + `read_ipc_stream()` and `write_ipc_stream()` are deprecated and ignored + (#49237). + +## New features + +- New `AzureFileSystem` class and `az_container()` helper for working with + Azure Blob Storage, analogous to `S3FileSystem` and `s3_bucket()`. Azure + support is enabled by default when building from source if libxml2 is + available, except on Windows (@marberts, #32123). Review Comment: The source-build condition is incomplete: `with_cloud_support()` disables Azure when curl or OpenSSL is unavailable as well as when libxml2 is unavailable (`r/tools/nixlibs.R:930-973`). Please list all three required libraries. ########## r/README.md: ########## @@ -56,15 +56,15 @@ tasks. It allows users to read and write data in a variety of formats: - Read and write Parquet files, an efficient and widely used columnar format -- Read and write Arrow (formerly known as Feather) files, a format optimized for speed and +- Read and write Arrow IPC (formerly known as Feather) files, a format optimized for speed and interoperability Review Comment: This conflates legacy Feather V1 with Arrow IPC. The project's format documentation says only Feather V2 is the Arrow IPC file format; describe it as “also known as Feather V2” so users do not infer that all Feather files are IPC files. ########## r/NEWS.md: ########## @@ -19,13 +19,43 @@ # arrow 25.0.1.9000 +## Breaking changes + +- `read_feather()` and `write_feather()` now warn that they are deprecated. Use + `read_ipc_file()` and `write_ipc_file()` instead. Similarly, + `format = "feather"` in `open_dataset()` and `write_dataset()` is deprecated + in favour of `format = "ipc"`, and extra arguments passed via `...` to + `read_ipc_stream()` and `write_ipc_stream()` are deprecated and ignored + (#49237). + +## New features + +- New `AzureFileSystem` class and `az_container()` helper for working with + Azure Blob Storage, analogous to `S3FileSystem` and `s3_bucket()`. Azure + support is enabled by default when building from source if libxml2 is + available, except on Windows (@marberts, #32123). + ## Minor improvements and fixes +- Variables with the same name as a function, such as `date`, can now be used + in dplyr verbs (#39688). +- R API requests made from parallel code are now thread-safe (#50239). Review Comment: This overstates the scope of #50239: the R API is not generally thread-safe. The fix delays parallel conversion tasks until R-side allocations and list assignments are complete, specifically removing the race during conversion to an R data frame. Please describe that bounded behavior instead. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
