[YUNIKORN-2756] Consider moving event_system#defaultEventChannelSize to configs#const

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 1.6.0
• Labels: newbie pull-request-available
• Target Version: 1.6.0
• Assignee: Chenchen Lai
• Reporter: Chia-Ping Tsai
• Created: 14/Jul/24 22:40
• Updated: 10/Sep/24 20:58
• Resolved: 12/Aug/24 15:22

Description

All other event-related configs are in configs#const[1], so we should move `defaultEventChannelSize`[0] make them together.

BTW, `defaultRingBufferSize`[2] will be removed by https://github.com/apache/yunikorn-core/pull/915, since its replacement is in configs#const already [3]

[0] https://github.com/apache/yunikorn-core/blob/f25bee90c2abd2c6682912dfdd0013ef2f4bc0ba/pkg/events/event_system.go#L37
[1] https://github.com/apache/yunikorn-core/blob/f25bee90c2abd2c6682912dfdd0013ef2f4bc0ba/pkg/common/configs/configs.go#L43
[2] https://github.com/apache/yunikorn-core/blob/f25bee90c2abd2c6682912dfdd0013ef2f4bc0ba/pkg/events/event_system.go#L38
[3] https://github.com/apache/yunikorn-core/blob/f25bee90c2abd2c6682912dfdd0013ef2f4bc0ba/pkg/common/configs/configs.go#L47

Issue Links

• links to: GitHub Pull Request #942