Skip to content

[SQL] Add version information to SQL compiler#6061

Merged
gz merged 1 commit intofeldera:mainfrom
mihaibudiu:issue4626
Apr 17, 2026
Merged

[SQL] Add version information to SQL compiler#6061
gz merged 1 commit intofeldera:mainfrom
mihaibudiu:issue4626

Conversation

@mihaibudiu
Copy link
Copy Markdown
Contributor

Fixes #4626

Until now the SQL compiler had no versioning information.
This PR brings it in line with the rest of the tools. A post-release script updates pom.xml, the maven build file.
The Java maven build uses a resource rewriting plugin to insert the information in file version.properties
The Java program can read the version at runtime using standard Java properties access.
The current version of the Java compiler is 0.287.0, and it will track the other tools.

Breaking changes

The compiler flag -h used to return an exit code of 1. Now it will return an exit code of 0.

@mihaibudiu mihaibudiu enabled auto-merge April 16, 2026 21:34
@mihaibudiu mihaibudiu requested review from abhizer and gz April 16, 2026 21:35
Copy link
Copy Markdown

@mythical-fred mythical-fred left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mihaibudiu mihaibudiu added this pull request to the merge queue Apr 16, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 16, 2026
@mihaibudiu mihaibudiu added this pull request to the merge queue Apr 16, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 17, 2026
</executions>
</plugin>
</plugins>
<!-- This substisutes the program.version from pom.xml to the resource file version.properties -->
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

substitutes

Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
@mihaibudiu mihaibudiu enabled auto-merge April 17, 2026 01:57
@mihaibudiu mihaibudiu added this pull request to the merge queue Apr 17, 2026
@gz gz removed this pull request from the merge queue due to a manual request Apr 17, 2026
@gz gz added this pull request to the merge queue Apr 17, 2026
@gz gz removed this pull request from the merge queue due to a manual request Apr 17, 2026
@gz gz added this pull request to the merge queue Apr 17, 2026
Merged via the queue into feldera:main with commit 94c6beb Apr 17, 2026
1 check passed
@gz gz deleted the issue4626 branch April 17, 2026 06:48
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.

[SQL] [Manager] give a warning if the SQL compiler version doesn't match the manager version

4 participants