]> source.dussan.org Git - gitea.git/commitdiff
Increase queue length (#27555) (#27562)
authorGiteabot <teabot@gitea.io>
Tue, 10 Oct 2023 12:22:26 +0000 (20:22 +0800)
committerGitHub <noreply@github.com>
Tue, 10 Oct 2023 12:22:26 +0000 (20:22 +0800)
Backport #27555 by @wolfogre

It should be OK to increase the default queue length since the default
type is "level".

IMO, the old default length (100) is a little too small. See
https://github.com/go-gitea/gitea/issues/27540#issuecomment-1754269491

IIRC, a larger length could lead to more memory usage only when the type
is "channel," but it's an obscure case. Otherwise, it's just a limit
(for "level" or "redis").

Co-authored-by: Jason Song <i@wolfogre.com>
custom/conf/app.example.ini
docs/content/administration/config-cheat-sheet.en-us.md
docs/content/administration/config-cheat-sheet.zh-cn.md
modules/queue/manager_test.go
modules/setting/queue.go

index 7db3c157c9dca9362fbc0d489fa3bae91ce234d3..4e2af7bbc0caa6e6acaa61c2acc6631029920c29 100644 (file)
@@ -1420,7 +1420,7 @@ LEVEL = Info
 ;DATADIR = queues/ ; Relative paths will be made absolute against `%(APP_DATA_PATH)s`.
 ;;
 ;; Default queue length before a channel queue will block
-;LENGTH = 100
+;LENGTH = 100000
 ;;
 ;; Batch size to send for batched queues
 ;BATCH_LENGTH = 20
index a337ae82a05be845c48fc775178f14ad2ad4535e..feea68f21225b5a0aada8308956a1a5ef4b47acc 100644 (file)
@@ -483,7 +483,7 @@ Configuration at `[queue]` will set defaults for queues with overrides for indiv
 
 - `TYPE`: **level**: General queue type, currently support: `level` (uses a LevelDB internally), `channel`, `redis`, `dummy`. Invalid types are treated as `level`.
 - `DATADIR`: **queues/common**: Base DataDir for storing level queues. `DATADIR` for individual queues can be set in `queue.name` sections. Relative paths will be made absolute against `%(APP_DATA_PATH)s`.
-- `LENGTH`: **100**: Maximal queue size before channel queues block
+- `LENGTH`: **100000**: Maximal queue size before channel queues block
 - `BATCH_LENGTH`: **20**: Batch data before passing to the handler
 - `CONN_STR`: **redis://127.0.0.1:6379/0**: Connection string for the redis queue type. For `redis-cluster` use `redis+cluster://127.0.0.1:6379/0`. Options can be set using query params. Similarly, LevelDB options can also be set using: **leveldb://relative/path?option=value** or **leveldb:///absolute/path?option=value**, and will override `DATADIR`
 - `QUEUE_NAME`: **_queue**: The suffix for default redis and disk queue name. Individual queues will default to **`name`**`QUEUE_NAME` but can be overridden in the specific `queue.name` section.
index 2849752c0ab5f10b6c766184ae8c65de467c8119..8248ed9a6aa0af68a3600a0b8e8e1ae94ab123e0 100644 (file)
@@ -472,7 +472,7 @@ menu:
 
 - `TYPE`:**level**:通用队列类型,当前支持:`level`(在内部使用 LevelDB)、`channel`、`redis`、`dummy`。无效的类型将视为 `level`。
 - `DATADIR`:**queues/common**:用于存储 level 队列的基本 DataDir。单独的队列的 `DATADIR` 可以在 `queue.name` 部分进行设置。相对路径将根据 `%(APP_DATA_PATH)s` 变为绝对路径。
-- `LENGTH`:**100**:通道队列阻塞之前的最大队列大小
+- `LENGTH`:**100000**:通道队列阻塞之前的最大队列大小
 - `BATCH_LENGTH`:**20**:在传递给处理程序之前批处理数据
 - `CONN_STR`:**redis://127.0.0.1:6379/0**:redis 队列类型的连接字符串。对于 `redis-cluster`,使用 `redis+cluster://127.0.0.1:6379/0`。可以使用查询参数来设置选项。类似地,LevelDB 选项也可以使用:**leveldb://relative/path?option=value** 或 **leveldb:///absolute/path?option=value** 进行设置,并将覆盖 `DATADIR`。
 - `QUEUE_NAME`:**_queue**:默认的 redis 和磁盘队列名称的后缀。单独的队列将默认为 **`name`**`QUEUE_NAME`,但可以在特定的 `queue.name` 部分中进行覆盖。
index 9a5b616b1a57f2a43cc302c12c19702e8ece2e95..15dd1b4f2f88ac3c89ff81d175a4834600e23e9b 100644 (file)
@@ -46,7 +46,7 @@ CONN_STR = redis://
        assert.Equal(t, "default", q.GetName())
        assert.Equal(t, "level", q.GetType())
        assert.Equal(t, filepath.Join(setting.AppDataPath, "queues/common"), q.baseConfig.DataFullDir)
-       assert.Equal(t, 100, q.baseConfig.Length)
+       assert.Equal(t, 100000, q.baseConfig.Length)
        assert.Equal(t, 20, q.batchLength)
        assert.Equal(t, "", q.baseConfig.ConnStr)
        assert.Equal(t, "default_queue", q.baseConfig.QueueFullName)
index fc15bd07ed68f5d6be728fe82164ea19f35ae831..251a6c1e305ccd385136a5ced148b6625c821103 100644 (file)
@@ -30,7 +30,7 @@ func GetQueueSettings(rootCfg ConfigProvider, name string) (QueueSettings, error
        queueSettingsDefault := QueueSettings{
                Type:    "level",         // dummy, channel, level, redis
                Datadir: "queues/common", // relative to AppDataPath
-               Length:  100,             // queue length before a channel queue will block
+               Length:  100000,          // queue length before a channel queue will block
 
                QueueName:   "_queue",
                SetName:     "_unique",