`river_job.max_attempts` (the database column) should have a default value of 25

オープン
#1,012 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
45/100
issue の種類
機能追加
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
go, postgresql
領域
database

調査の方向性

Start by locating the river_job schema and the next migration that changes it. Check how other columns define defaults, then verify that a raw SQL insert can omit max_attempts and receive 25; the change is done when the schema default is included in that migration.

索引モデルが issue の本文から書いたものです。

説明

needs db migration

While writing documentation for how to insert a job using raw SQL, I came across the fact that while almost every river_job field gets a reasonable default, max_attempts is the odd man out and must have a value specified despite having a clear default of 25 in code.

Having a default on this column would be a slight improvement in that it'd make a manually inserted job a little easier to insert. Also, I think not having a default on this property may have been a mistake anyway since all other properties except the highly variable ones like kind and args have one.

Not worth its own migration, but let's ship the change opportunistically during the next migration that goes out.

主要言語
Go
スター
5.7k
フォーク
179
平均マージ
15時間 43分
マージ済み PR（30日）
13

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

riverqueue/river のほかの issue

riverqueue/river の issue をすべて見る

似ている issue

Go の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。