Skip to content

Use entire shadow jar to fix IT#3447

Merged
penghuo merged 9 commits intoopensearch-project:mainfrom
xinyual:useEntireShadowJar
Mar 25, 2025
Merged

Use entire shadow jar to fix IT#3447
penghuo merged 9 commits intoopensearch-project:mainfrom
xinyual:useEntireShadowJar

Conversation

@xinyual
Copy link
Copy Markdown
Contributor

@xinyual xinyual commented Mar 19, 2025

Description

We use entire jar to fix security issue in opensearch-project/job-scheduler#738

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • New functionality has javadoc added.
  • New functionality has a user manual doc added.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

penghuo and others added 2 commits March 19, 2025 19:02
Signed-off-by: Peng Huo <penghuo@gmail.com>
Signed-off-by: xinyual <xinyual@amazon.com>
@LantaoJin
Copy link
Copy Markdown
Member

@xinyual could you make it draft and convert to ready when CI failure resolved.

@xinyual xinyual marked this pull request as draft March 20, 2025 02:42
@xinyual xinyual changed the base branch from feature/calcite-engine to main March 23, 2025 13:35
@xinyual xinyual marked this pull request as ready for review March 23, 2025 13:38
Signed-off-by: xinyual <xinyual@amazon.com>
@LantaoJin
Copy link
Copy Markdown
Member

To unblock #3468, #3468 will continue disable remote ITs. Please enable them in this PR by removing the commented code here.

@xinyual xinyual force-pushed the useEntireShadowJar branch from 49f5254 to 01d7d11 Compare March 25, 2025 05:53
xinyual added 4 commits March 25, 2025 13:58
Signed-off-by: xinyual <xinyual@amazon.com>
Signed-off-by: xinyual <xinyual@amazon.com>
Signed-off-by: xinyual <xinyual@amazon.com>
Signed-off-by: xinyual <xinyual@amazon.com>
Copy link
Copy Markdown
Collaborator

@penghuo penghuo left a comment

Choose a reason for hiding this comment

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

Thx!

@penghuo penghuo merged commit ccae017 into opensearch-project:main Mar 25, 2025
22 checks passed
penghuo added a commit that referenced this pull request Jun 16, 2025
---------

Signed-off-by: Peng Huo <penghuo@gmail.com>
Signed-off-by: xinyual <xinyual@amazon.com>
Co-authored-by: Peng Huo <penghuo@gmail.com>
Signed-off-by: xinyual <xinyual@amazon.com>
@xinyual xinyual mentioned this pull request Dec 12, 2025
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

calcite calcite migration releated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants