commit is forbidden in queue jobs
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 18/100
調査の方向性
The issue names no source files, only the pattern env.cr.commit() in delayed job functions and a reference to OCA/queue pull request 880, which is not linked here. Start by reading that pull request and the 22-comment thread to see what has been decided about commits in jobs. The issue asks for feedback rather than a change, so there is no clear definition of done yet; check with maintainers before writing code.
索引モデルが issue の本文から書いたものです。
説明
It is forbidden to call env.cr.commit() in delayed job functions.
It has always been bad practice, but now a commit releases the job lock and can cause it to be started again (while still running) by the dead jobs requeuer.
https://github.com/OCA/queue/pull/880 makes such commits an error to make that explicit.
I open this issue to collect feedback on this, as there might be (semi-)legitimate uses of commit.
- 主要言語
- Python
- スター
- 271
- フォーク
- 602
- 平均マージ
- 4日 4時間
- マージ済み PR（30日）
- 6
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
OCA/queue のほかの issue
-
enhancement
難易度 3/5 1〜2日 初心者へのやさしさ 30/100
メンテナーはふだん 1 日以内に返信
-
help wanted no stale work in progress
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
メンテナーはふだん 1 日以内に返信
-
Delaying unlink or write on a model whose method was replaced raises AttributeError at enqueue time対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンbug
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
メンテナーはふだん 1 日以内に返信
-
[19.0] queue_job: Jobs stay pending after the jobrunner's database connection is silently dropped対応中かも @HoloborodkoBohdan が 32 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 25/100
OCA/queue#984 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
getchaintips leaves out the active tip once a header is staged on it, where Core always lists itオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
btclib-org/btclib-node#2158 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
dbt-msft/dbt-sqlserver#897 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
Gathin23/cleanyourtv#2770 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
vllm-project/vllm#64845 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
dekobon/big-code-analysis#1729 ·
メンテナーはふだん 1 日以内に返信