Skip to content
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

[improve][broker] Improve exception for topic does not have schema to check #22974

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

shibd
Copy link
Member

@shibd shibd commented Jun 25, 2024

Motivation

If a consumer use schema to connector a active topic with empty(or bytes) schema, will receive IncompatibleSchemaException: Topic does not have schema to check

But, This exception message may confuse for users.

Modifications

  • Change exception msg to: Failed to add schema to an active topic with empty(BYTES) schema: new schema type schema.getType()

Verifying this change

  • testAutoProduceAndSpecifiedConsumer will cover it.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

@shibd shibd added type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages ready-to-test release/3.3.1 release/3.0.6 labels Jun 25, 2024
@shibd shibd self-assigned this Jun 25, 2024
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jun 25, 2024
@shibd shibd closed this Jun 25, 2024
@shibd shibd merged commit 4c84788 into apache:master Jul 1, 2024
51 of 52 checks passed
shibd added a commit that referenced this pull request Jul 1, 2024
shibd added a commit that referenced this pull request Jul 1, 2024
shibd added a commit that referenced this pull request Jul 1, 2024
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Jul 10, 2024
… check (apache#22974)

(cherry picked from commit 4c84788)
(cherry picked from commit bbb37b3)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Jul 15, 2024
… check (apache#22974)

(cherry picked from commit 4c84788)
(cherry picked from commit bbb37b3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-picked/branch-3.0 cherry-picked/branch-3.2 cherry-picked/branch-3.3 doc-not-needed Your PR changes do not impact docs ready-to-test release/3.0.6 release/3.2.4 release/3.3.1 type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants