Skip to content

OBSDATA-9960: Revert "Modify "maxRowsInMemory"and "maxBytesInMemory" to use values …#325

Merged
Suraj Goel (suraj-goel) merged 1 commit into30.0.1-confluentfrom
sugoel/revert-persist-config-overrides
Apr 24, 2025
Merged

OBSDATA-9960: Revert "Modify "maxRowsInMemory"and "maxBytesInMemory" to use values …#325
Suraj Goel (suraj-goel) merged 1 commit into30.0.1-confluentfrom
sugoel/revert-persist-config-overrides

Conversation

@suraj-goel
Copy link
Copy Markdown
Member

…from config (#323)"

This reverts commit 34c3446.

Fixes #XXXX.

Description

Fixed the bug ...

Renamed the class ...

Added a forbidden-apis entry ...

Release note

For tips about how to write a good release note, see Release notes.


Key changed/added classes in this PR
  • MyFoo
  • OurBar
  • TheirBaz

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

@suraj-goel Suraj Goel (suraj-goel) marked this pull request as ready for review April 24, 2025 09:39
@suraj-goel Suraj Goel (suraj-goel) requested a review from a team as a code owner April 24, 2025 09:39
@suraj-goel Suraj Goel (suraj-goel) enabled auto-merge (squash) April 24, 2025 09:39
@suraj-goel Suraj Goel (suraj-goel) merged commit 5d7cf7e into 30.0.1-confluent Apr 24, 2025
2 checks passed
@suraj-goel Suraj Goel (suraj-goel) deleted the sugoel/revert-persist-config-overrides branch April 24, 2025 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants