Skip to content

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

Merged
vlsi merged 1 commit intoapache:masterfrom
vlsi:jandex_verify_by_default
Jan 25, 2021
Merged

[CALCITE-4459] Verify the bytecode with Jandex by default#2334
vlsi merged 1 commit intoapache:masterfrom
vlsi:jandex_verify_by_default

Conversation

@vlsi
Copy link
Contributor

@vlsi vlsi commented Jan 24, 2021

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

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants