Skip to content

[fix][broker] fixed the build error for pattern matching variable in lower JVM versions - #19362

Merged
merlimat merged 1 commit into
apache:branch-2.10from
heesung-sohn:branch-2.10-fix-pattern-matching
Jan 30, 2023
Merged

[fix][broker] fixed the build error for pattern matching variable in lower JVM versions#19362
merlimat merged 1 commit into
apache:branch-2.10from
heesung-sohn:branch-2.10-fix-pattern-matching

Conversation

@heesung-sohn

Copy link
Copy Markdown
Contributor

Motivation

Fix the build error from the pattern-matching variable introduced in #19343.

the pattern-matching variable is available for java >=14.

Modifications

Removed the pattern-matching variable.

Verifying this change

  • Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Documentation

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

Matching PR in forked repository

PR in forked repository: heesung-sohn#25

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Jan 30, 2023
@merlimat
merlimat merged commit 0dfbc61 into apache:branch-2.10 Jan 30, 2023
@heesung-sohn

Copy link
Copy Markdown
Contributor Author

Please let me know if we need a separate PR to merge it to branch-2.11.

@merlimat

Copy link
Copy Markdown
Contributor

2.11 is already on Java 17, so we don't need to backport there.

nicoloboschi pushed a commit to datastax/pulsar that referenced this pull request Feb 28, 2023
@heesung-sohn
heesung-sohn deleted the branch-2.10-fix-pattern-matching branch April 2, 2024 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants