From 2af5ffb04945f1edae16f843924db7eb633fa5b7 Mon Sep 17 00:00:00 2001
From: eitsupi <50911393+eitsupi@users.noreply.github.com>
Date: Sun, 2 Jun 2024 22:47:51 +0900
Subject: [PATCH] feat!: don't rechunk by default in `pl$concat()` (#1125)

---
 NEWS.md              | 1 +
 R/functions__eager.R | 2 +-
 man/pl_concat.Rd     | 2 +-
 3 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/NEWS.md b/NEWS.md
index 4f162678b..85259e4b6 100644
--- a/NEWS.md
+++ b/NEWS.md
@@ -50,6 +50,7 @@
   (#1112).
 - In `$dt$combine()`, the arguments `tm` and `tu` are renamed `time` and
   `time_unit` (#1116).
+- The default value of the `rechunk` argument of `pl$concat()` is changed from `TRUE` to `FALSE` (#1125).
 
 ### New features
 
diff --git a/R/functions__eager.R b/R/functions__eager.R
index 8c707d2e2..fc264e0d5 100644
--- a/R/functions__eager.R
+++ b/R/functions__eager.R
@@ -59,7 +59,7 @@ pl_concat = function(
       "vertical", "vertical_relaxed", "horizontal",
       "diagonal", "diagonal_relaxed"
     ),
-    rechunk = TRUE,
+    rechunk = FALSE,
     parallel = TRUE) {
   l = unpack_list(..., skip_classes = "data.frame", .context = "in pl$concat")
 
diff --git a/man/pl_concat.Rd b/man/pl_concat.Rd
index ab1ceef83..e56719897 100644
--- a/man/pl_concat.Rd
+++ b/man/pl_concat.Rd
@@ -7,7 +7,7 @@
 pl_concat(
   ...,
   how = c("vertical", "vertical_relaxed", "horizontal", "diagonal", "diagonal_relaxed"),
-  rechunk = TRUE,
+  rechunk = FALSE,
   parallel = TRUE
 )
 }