remove shadow jar

[xinyual]

Description

We try to remove shadow jar since access control now is already removed in core.
Revert the change in PR #3447

Also fix an IT due to the import name changed from
shaded.com.google -> com.google

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.
• New PPL command checklist all confirmed.
• API changes companion pull request created.
• Commits are signed per the DCO using --signoff or -s.
• 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.

[coderabbitai]

📝 Walkthrough

Summary by CodeRabbit

• Chores
  • Simplified plugin build and packaging by removing the Shadow plugin integration and all related packaging logic, including ZIP manipulation and artifact relocation handling. This streamlines the overall build process.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

This PR simplifies Gradle build configuration by removing Shadow plugin integration and related artifact packaging logic, while correcting quote escaping in a test expectations file. The changes eliminate specialized ZIP manipulation, JAR filtering, and publication hooks previously associated with shadow-based packaging.

Changes

Cohort / File(s)	Summary
Test expectations integ-test/src/test/resources/expectedOutput/ppl/explain_patterns_simple_pattern_agg_push.yaml	Quote escaping corrections within embedded script string; no functional changes
Build configuration plugin/build.gradle	Removed Shadow plugin integration: eliminated plugin application, shadowJar task configuration, publication wiring, pre/post publish hooks, and JAR filtering/relocation logic

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

• Verify build completes successfully without Shadow plugin and packaging produces expected output
• Confirm no other modules or publication workflows depend on shadow artifacts or relocated JARs
• Validate JAR relocation behavior is no longer needed or handled elsewhere

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'remove shadow jar' directly and concisely describes the main change in the pull request, which is removing shadow jar packaging logic from the Gradle configuration.
Description check	✅ Passed	The description clearly explains the purpose (removing shadow jar because access control was removed in core), references the PR being reverted, and mentions the related IT fix for import name changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bcfcd00 and e3153a3.

📒 Files selected for processing (2)

• integ-test/src/test/resources/expectedOutput/ppl/explain_patterns_simple_pattern_agg_push.yaml (1 hunks)
• plugin/build.gradle (0 hunks)

💤 Files with no reviewable changes (1)

• plugin/build.gradle

🔇 Additional comments (1)

integ-test/src/test/resources/expectedOutput/ppl/explain_patterns_simple_pattern_agg_push.yaml (1)

8-19: Escaping change looks consistent, but golden output is very brittle—validate YAML+JSON parsing and line continuations.
Given the triple-escaping (YAML string containing JSON containing an escaped serialized blob), please ensure this still parses as a single request: string and matches the actual IT output exactly (especially the \\\ line continuations around Line 12-13 and trailing quoting near Line 18-19).

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[LantaoJin]

It confused me why a plan changed by your PR

[xinyual]

After decode, the script previously using
import shaded.com.google...
The shaded here is caused by shade jar.
But now we remove it, so it is
import com.google...
So the script changed.

[LantaoJin]

can you run explain command with extended and attach the result here

[LantaoJin]

oh, it's a v2 plan. not sure how to decode the base64 string.

[xinyual]

Using online decode and here are the result:
using shadow jar

��sr�6org.opensearch.sql.expression.parse.PatternsExpressionΪG����Z��useCustomPatternL��patternt��Ljava/util/regex/Pattern;xr�3org.opensearch.sql.expression.parse.ParseExpression
}���L�
identifiert�Lorg/opensearch/sql/expression/Expression;L�
identifierStrt��Ljava/lang/String;L��patternq���L��sourceFieldq���xr�0org.opensearch.sql.expression.FunctionExpression0uj{���L� argumentst��Ljava/util/List;L�functionNamet�5Lorg/opensearch/sql/expression/function/FunctionName;xpsr�=shaded.com.google.common.collect.ImmutableList$SerializedForm�����������[��elementst��[Ljava/lang/Object;xpur��[Ljava.lang.Object;X�s)l���xp����sr�1org.opensearch.sql.expression.ReferenceExpressionD\�����L��attrq���L��pathsq���L��rawPathq���L��typet�'Lorg/opensearch/sql/data/type/ExprType;xpt��emailsr��java.util.Arrays$ArrayList٤<͈����[��aq��
xpur��[Ljava.lang.String;V�{G���xp����q���q���sr�:org.opensearch.sql.opensearch.data.type.OpenSearchTextType�1D���L��fieldst��Ljava/util/Map;xr�:org.opensearch.sql.opensearch.data.type.OpenSearchDataTypec��5���L�exprCoreTypet�+Lorg/opensearch/sql/data/type/ExprCoreType;L��mappingTypet�HLorg/opensearch/sql/opensearch/data/type/OpenSearchDataType$MappingType;L�
propertiesq���xpr�)org.opensearch.sql.data.type.ExprCoreType�����������xr��java.lang.Enum�����������xpt��UNKNOWNr�Forg.opensearch.sql.opensearch.data.type.OpenSearchDataType$MappingType�����������xq���t��Textsr�<*shaded.com.google.common.collect.ImmutableMap$SerializedForm*�����������L��keyst��Ljava/lang/Object;L��valuesq��$xpuq������uq������sr��java.util.CollSerW:�����I��tagxp����w�����t��keywordsq���q���t��STRINGq�� t��Keywordq��%xsr�/org.opensearch.sql.expression.LiteralExpressionEB-ǂ$���L� exprValuet�)Lorg/opensearch/sql/data/model/ExprValue;xpsr�-org.opensearch.sql.data.model.ExprStringValue�A2%s�����L��valueq���xr�/org.opensearch.sql.data.model.AbstractExprValuekv��D���xpt��sq��0sq��3t��patterns_fieldsr�3org.opensearch.sql.expression.function.FunctionName�8Mg���L�functionNameq���xpt��patternsq��7q��9q��2q�~���p

And without shadow jar

��sr�6org.opensearch.sql.expression.parse.PatternsExpressionΪG����Z��useCustomPatternL��patternt��Ljava/util/regex/Pattern;xr�3org.opensearch.sql.expression.parse.ParseExpression}���L�
identifiert�Lorg/opensearch/sql/expression/Expression;L�
identifierStrt��Ljava/lang/String;L��patternq���L��sourceFieldq���xr�0org.opensearch.sql.expression.FunctionExpression0uj{���L� argumentst��Ljava/util/List;L�functionNamet�5Lorg/opensearch/sql/expression/function/FunctionName;xpsr�6*com.google.common.collect.ImmutableList$SerializedForm�����������[��elementst��[Ljava/lang/Object;xpur��[Ljava.lang.Object;X�s)l���xp����sr�1org.opensearch.sql.expression.ReferenceExpressionD\�����L��attrq���L��pathsq���L��rawPathq���L��typet�'Lorg/opensearch/sql/data/type/ExprType;xpt��emailsr��java.util.Arrays$ArrayList٤<͈����[��aq��
xpur��[Ljava.lang.String;V�{G���xp����q���q���sr�:org.opensearch.sql.opensearch.data.type.OpenSearchTextType�1D���L��fieldst��Ljava/util/Map;xr�:org.opensearch.sql.opensearch.data.type.OpenSearchDataTypec��5���L�exprCoreTypet�+Lorg/opensearch/sql/data/type/ExprCoreType;L��mappingTypet�HLorg/opensearch/sql/opensearch/data/type/OpenSearchDataType$MappingType;L�
propertiesq���xpr�)org.opensearch.sql.data.type.ExprCoreType�����������xr��java.lang.Enum�����������xpt��UNKNOWNr�Forg.opensearch.sql.opensearch.data.type.OpenSearchDataType$MappingType�����������xq���t��Textsr�5com.google.common.collect.ImmutableMap$SerializedForm*�����������L��keyst��Ljava/lang/Object;L��valuesq��$xpuq������uq������sr��java.util.CollSerW:�����I��tagxp����w�����t��keywordsq���q���t��STRINGq�� t��Keywordq��%xsr�/org.opensearch.sql.expression.LiteralExpressionEB-ǂ$���L� exprValuet�)Lorg/opensearch/sql/data/model/ExprValue;xpsr�-org.opensearch.sql.data.model.ExprStringValue�A2%s�����L��valueq���xr�/org.opensearch.sql.data.model.AbstractExprValuekv��D���xpt��sq��0sq��3t��patterns_fieldsr�3org.opensearch.sql.expression.function.FunctionName�8Mg���L�functionNameq���xpt��patternsq��7q��9q��2q�~���p

You could see the com.google.common.collect.ImmutableList$SerializedForm path changed.