-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: --experimental-use-server-side-upsert document #983
Conversation
|
||
We have implemented the `--experimental-use-server-side-upsert` option, which executes the record import command using [the upsert option of the bulk record update API](https://cybozu.dev/ja/kintone/docs/rest-api/records/update-records/) | ||
|
||
Previously, cli-kintone implemented its own upsert mode, but by using `--experimental-use-server-side-upsert`, there is a possibility that the performance of the record import command will improve. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously, cli-kintone implemented its own upsert mode,
↓
以前からあるcli-kintone独自のupsert機能があります、という意図の表現でしょうか?
Cli-kintone has been running with its own upsert mode
やCli-kintone has had its own upsert mode
が自然に聞こえるかもです。
(そもそも主語がcli-kintoneなので、人が主語となるべきimplementの動詞とマッチしていなさそう)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hisasami
なるほど、ご提案いただいた文章に修正してみます!
ありがとうございます〜
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2箇所コメントしました!
--update-key レコード番号 \ | ||
--experimental-use-server-side-upsert | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
安定性が確認できたらデフォルトオンにする予定というのも書いておいて良さそうです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tasshi-me
ありがとうございますー
一文追加しました。
855f943
Co-authored-by: tasshi / Masaharu Tashiro <[email protected]>
Why
What
--experimental-use-server-side-upsert
How to test
Checklist
pnpm lint
andpnpm test
on the root directory.