Skip to content

[CALCITE-4459] Verify the bytecode with Jandex by default #2334

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 25, 2021
Merged

[CALCITE-4459] Verify the bytecode with Jandex by default #2334

merged 1 commit into from
Jan 25, 2021
+3 −2

Conversation

vlsi
Copy link
Contributor

@vlsi vlsi commented Jan 24, 2021 •
edited
Loading

Here's the relevant jandex release notice: smallrye/jandex#104 (comment)

@vlsi vlsi changed the title [CALCITE-4470] Verify the bytecode with Jandex by default [CALCITE-4459] Verify the bytecode with Jandex by default Jan 24, 2021
@vlsi vlsi force-pushed the jandex_verify_by_default branch from 0a46f0c to 801f33d Compare January 24, 2021 20:23
@vlsi vlsi added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Jan 24, 2021
@amaliujia
Copy link
Contributor

+1
@vlsi thanks for taking care of this!

@vlsi vlsi merged commit 801f33d into apache:master Jan 25, 2021
@vlsi vlsi deleted the jandex_verify_by_default branch January 25, 2021 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
LGTM-will-merge-soon Overall PR looks OK. Only minor things left.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vlsi @amaliujia