GitHub user xintongsong added a comment to the discussion: [Feature] Per-Event-Type Configurable Log Levels for Event Log
@weiqingy One more minor comments. I think we should add some infix between `event-log` and the event type in the config keys. Something like `event-log.type.<EVENT_TYPE>.level`. With #424, technically `<EVENT_TYPE>` can be anything. It can become ambiguous that `event-log.standard.xxx` can either refers to built-in config options, or a group of event types with the common prefix `standard`. Adding a `type` infix would help avoid that ambiguous. Other than that, LGTM. I think we can move to the implementation. GitHub link: https://github.com/apache/flink-agents/discussions/552#discussioncomment-16008666 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected]
