8000 Ignore me: by tabVersion · Pull Request #21868 · risingwavelabs/risingwave · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Ignore me: #21868

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 5 commits into from
Closed

Ignore me: #21868

wants to merge 5 commits into from

Conversation

tabVersion
Copy link
Contributor
@tabVersion tabVersion commented May 14, 2025

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.

two pr ahead of release-2.2

#21857
#21865

#21900

What's changed and what's your intention?

Checklist

  • I have written necessary rustdoc comments.
  • I have added necessary unit tests and integration tests.
  • I have added test labels as necessary.
  • I have added fuzzing tests or opened an issue to track them.
  • My PR contains breaking changes.
  • My PR changes performance-critical code, so I will run (micro) benchmarks and present the results.
  • I have checked the Release Timeline and Currently Supported Versions to determine which release branches I need to cherry-pick this PR into.

Documentation

  • My PR needs documentation updates.
Release note

tab added 3 commits May 14, 2025 13:29
- Introduced a new function `read_kafka_log_level` to read the log level from the environment variable `RISINGWAVE_KAFKA_LOG_LEVEL`.
- Updated `KafkaConnection`, `KafkaSinkWriter`, and `KafkaSplitReader` to utilize the dynamic log level configuration.
- Ensured that an invalid log level falls back to the default level of INFO with a warning log.
tab added 2 commits May 16, 2025 22:20
fix
Signed-off-by: tab <tabversion@bupt.icu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0