Skip to content

Comments

[INLONG-12071][Manager] Fix the incorrect filtering logic that checks whether messages match#12072

Open
healchow wants to merge 2 commits intoapache:masterfrom
healchow:heal-dev
Open

[INLONG-12071][Manager] Fix the incorrect filtering logic that checks whether messages match#12072
healchow wants to merge 2 commits intoapache:masterfrom
healchow:heal-dev

Conversation

@healchow
Copy link
Member

Fixes #12071

Motivation

Fix the incorrect filtering logic that checks whether messages match, such as like puzzy match and is null condition are incorrectly.

Modifications

Enhance checkIfFilter method in DeserializeOperator class.

Verifying this change

(Please pick either of the following options)

  • This change is a trivial rework/code cleanup without any test coverage.

  • This change is already covered by existing tests, such as:
    (please describe tests)

  • This change added tests and can be verified as follows:

    • Added unit tests for modified codes

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

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.

[Bug][Manager] Querying MQ messages returned incorrect results from the LIKE matching operator

1 participant