Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

aogzpa
Copy link

@aogzpa aogzpa commented Feb 7, 2025

What

Fixed the windows batch file in a way that Confluent Community Edition schema-registry works without docker. There are only symlinks—instead of the actual JARs for the needed dependencies—in the folders added to the :concat jump goto label that fills the %CLASSPATH%. Thus, this change adds the missing classes that are in the ksqldb directory solving this issue.

See also: confluentinc/kafka#1581

Checklist

  • Contains customer facing changes? Including API/behavior changes
  • Is this change gated behind feature flag(s)?
    • List the LD flags needed to be set to enable this change
  • Did you add sufficient unit test and/or integration test coverage for this PR?
    • If not, please explain why it is not required
  • Does this change require modifying existing system tests or adding new system tests?
    • If so, include tracking information for the system test changes

Test & Review

Before:
image

After
image

@aogzpa aogzpa requested a review from a team as a code owner February 7, 2025 09:18
@confluent-cla-assistant
Copy link

confluent-cla-assistant bot commented Feb 7, 2025

🎉 All Contributor License Agreements have been signed. Ready to merge.
✅ aogzpa
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

@aogzpa aogzpa changed the title fix(bin): add missing JAR directory for windows batch script fix(bin): add missing JAR directory to windows batch script Feb 7, 2025
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.