Skip to content

Commit

Permalink
chore: remove --batch-size
Browse files Browse the repository at this point in the history
  • Loading branch information
rozenmd committed Dec 10, 2024
1 parent 9a1063b commit 5bb4864
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 11 deletions.
9 changes: 9 additions & 0 deletions .changeset/major-remove-d1-deprecated-batch-size.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
---
"wrangler": major
---

Remove `--batch-size` as an option for `wrangler d1 execute` and `wrangler d1 migrations apply`

BREAKING CHANGE: This change removes the deprecated `--batch-size` flag, as it is no longer necessary to decrease the number of queries wrangler sends to D1.

Closes #7470
6 changes: 0 additions & 6 deletions packages/wrangler/src/d1/execute.ts
Original file line number Diff line number Diff line change
Expand Up @@ -81,12 +81,6 @@ export function Options(yargs: CommonYargsArgv) {
describe: "Execute commands/files against a preview D1 DB",
type: "boolean",
default: false,
})
.option("batch-size", {
describe: "Number of queries to send in a single batch",
type: "number",
deprecated: true,
hidden: true,
});
}

Expand Down
6 changes: 1 addition & 5 deletions packages/wrangler/src/d1/migrations/apply.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,7 @@ import type {
} from "../../yargs-types";

export function ApplyOptions(yargs: CommonYargsArgv) {
return MigrationOptions(yargs).option("batch-size", {
describe: "Number of queries to send in a single batch",
type: "number",
deprecated: true,
});
return MigrationOptions(yargs);
}

type ApplyHandlerOptions = StrictYargsOptionsToInterface<typeof ApplyOptions>;
Expand Down

0 comments on commit 5bb4864

Please sign in to comment.