Skip to content

[fix][broker] Fix dispatcherPauseOnAckStatePersistentEnabled and schemaValidationEnforced returning wrong values when applied=true - #26472

Open
void-ptr974 wants to merge 2 commits into
apache:masterfrom
void-ptr974:fix-ack-state-pause-applied-policy
Open

[fix][broker] Fix dispatcherPauseOnAckStatePersistentEnabled and schemaValidationEnforced returning wrong values when applied=true#26472
void-ptr974 wants to merge 2 commits into
apache:masterfrom
void-ptr974:fix-ack-state-pause-applied-policy

Conversation

@void-ptr974

@void-ptr974 void-ptr974 commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Problem

Two topic policy queries could return the wrong effective value when
applied=true:

  • dispatcherPauseOnAckStatePersistentEnabled returned false when it was
    enabled at the namespace or broker level and was not set on the topic.
  • schemaValidationEnforced returned false for an explicit topic value of
    false even when the broker setting was true and forced schema validation
    at runtime.

Fix

  • Resolve the applied dispatcher pause value from topic, namespace, and broker
    settings while preserving explicit false overrides.
  • Include broker-level schema validation in the applied topic value while still
    allowing topic false to override namespace true when the broker setting is
    disabled.
  • Keep applied=false, remove behavior, and local/global policy scopes
    unchanged.
  • Keep policy lookups asynchronous without loading topics.
  • Add regression tests for inheritance, overrides, unset values, and query
    modes.

Tests

  • AckStatePausePolicyTest: 9 passed
  • AdminApiSchemaValidationEnforcedTest: 7 passed
  • ./gradlew quickCheck: passed

Resolve inherited namespace and broker values when querying the dispatcher pause-on-ack-state-persistence policy with applied=true, while preserving explicit false values and raw-query semantics.

Assisted-by: Codex (GPT-5)
@void-ptr974 void-ptr974 changed the title [fix][broker] Fix dispatcherPauseOnAckStatePersistentEnabled returning false when enabled at namespace or broker level [fix][broker] Fix wrong applied values for dispatcherPauseOnAckStatePersistentEnabled and schemaValidationEnforced Sep 6, 2026
@void-ptr974 void-ptr974 changed the title [fix][broker] Fix wrong applied values for dispatcherPauseOnAckStatePersistentEnabled and schemaValidationEnforced [fix][broker] Fix dispatcherPauseOnAckStatePersistentEnabled and schemaValidationEnforced returning wrong values when applied=true Sep 6, 2026
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.

1 participant