Skip to content

发件箱:积压收不动时日志没有限流,隔离那一行打了两遍 #138

Open

Description

@joesmart

背景

#131(Closes #115)给发件箱加了总量上限,复核时发现两处日志问题,都不挡合并。

1. 积压收不动时日志没有限流

条件是:积压中收不动的帧超过上限(默认 128 MiB),并且仍有新的可靠上行进来。满足时,每次 flush 都会打两行「收短了…」和「收短后仍有 N 字节…照收」。flush 最多每 100ms 一次,所以最多大约每秒 20 行。

  • 桌面端:只把这些行转发到自己的 stdout,不写文件,影响可以忽略。
  • 云电脑:docker 容器日志如果不轮转,会一直变大。

建议这两行只在两个时刻各打一次:积压越过上限时,以及回落到线下时。

2. 隔离那一行打了两遍

reportQuarantine 原来就会写一次 console.error。#131 之后,发件箱日志也走默认的 stdout,所以隔离这一行在 stderr 和 stdout 各出现一次。去掉其中一处即可。

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions