Skip to content

Conversation

@asifabashar
Copy link
Owner

Description

[Describe what this change achieves]

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.

@asifabashar asifabashar added the enhancement New feature or request label Jan 26, 2026
asifabashar and others added 2 commits January 27, 2026 10:55
…oject#4786 (opensearch-project#5011)

* transpose command implementation

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* transpose rows to columns

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added argument type missing map and hashmap

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added more validations

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added validation

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* index.md formatting fix

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* doc format

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* coderabbit review fixes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added recommended changes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added recommended changes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* trim columnName

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* per review moved to class varialble.

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* per review moved to class varialble.

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added field resolution

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* fix by removing metadata field

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* fixed explain test after removing of metadata fields in transpose result

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

---------

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
…#5075)

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
@asifabashar asifabashar force-pushed the feature-fieldformat branch 2 times, most recently from 8a53491 to 5d2721f Compare January 27, 2026 23:00
* MvCombine Command Feature

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* MvCombine Command Feature

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Add doctests to MvCombine

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* spotlesscheck apply

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* spotlesscheck apply

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* spotlesscheck apply

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* spotlessapply

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address coderrabbit comments

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address coderrabbit comments

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address coderrabbit comments

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address coderrabbit comments

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address coderrabbit comments

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address coderrabbit comments

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Add mvcombine to index.md

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Remove the nomv related implementation as that command is still not yet implemented

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Remove the nomv related implementation as that command is still not yet implemented

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Remove the nomv related implementation as that command is still not yet implemented

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Remove the nomv related implementation as that command is still not yet implemented

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* complete the checklist from ppl-commands.md

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* spotlessApply

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Add visitMvCombine method to the FieldResolutionVisitor

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Apply spotlesscheck

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Add changes to exclude the metadata fields and remove the CAST logic

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address CrossClusterSearchIT comment

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address CrossClusterSearchIT comment

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address CrossClusterSearchIT comment

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Coderrabbit issues

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Coderrabbit issues

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Coderrabbit issues

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Coderrabbit issues

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address comments

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

* Address comments

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>

---------

Signed-off-by: Srikanth Padakanti <srikanth_padakanti@apple.com>
Signed-off-by: Srikanth Padakanti <srikanth29.9@gmail.com>
Co-authored-by: Srikanth Padakanti <srikanth_padakanti@apple.com>
@asifabashar asifabashar force-pushed the feature-fieldformat branch 2 times, most recently from d04c315 to 78d8956 Compare January 28, 2026 18:18
yuancu and others added 5 commits January 29, 2026 16:50
…terSearchIT (opensearch-project#5085)

* Move tests for calcite-only commands from CrossClusterSearchIT to CalciteCrossClusterSearchIT

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

* Ask to use CalciteCrossClusterSearchIT when developing calcite-only commands

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

---------

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* 3.5 release notes

Signed-off-by: Eric Wei <mengwei.eric@gmail.com>

* move a few notes

Signed-off-by: Eric Wei <mengwei.eric@gmail.com>

---------

Signed-off-by: Eric Wei <mengwei.eric@gmail.com>
…r.doPrivilegedChecked (opensearch-project#5061)

Co-authored-by: Joshua Li <joshuali925@gmail.com>
* Fix the filter of integTestWithSecurity

Signed-off-by: Lantao Jin <ltjin@amazon.com>

* Fix IT

Signed-off-by: Lantao Jin <ltjin@amazon.com>

* Fix IT

Signed-off-by: Lantao Jin <ltjin@amazon.com>

* refactor

Signed-off-by: Lantao Jin <ltjin@amazon.com>

* fix IT

Signed-off-by: Lantao Jin <ltjin@amazon.com>

* Use Junit4

Signed-off-by: Lantao Jin <ltjin@amazon.com>

* fix

Signed-off-by: Lantao Jin <ltjin@amazon.com>

* fix httpclient leaking

Signed-off-by: Lantao Jin <ltjin@amazon.com>

* Fix permission IT in Junit4

Signed-off-by: Lantao Jin <ltjin@amazon.com>

---------

Signed-off-by: Lantao Jin <ltjin@amazon.com>
)

* [CVE] Upgrade assertj-core to 3.27.7

Signed-off-by: Jialiang Liang <jiallian@amazon.com>

* update release not of 3.5

Signed-off-by: Jialiang Liang <jiallian@amazon.com>

---------

Signed-off-by: Jialiang Liang <jiallian@amazon.com>
Swiddis and others added 14 commits February 3, 2026 13:26
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

# Conflicts:
#	ppl/src/main/antlr/OpenSearchPPLLexer.g4
#	ppl/src/main/antlr/OpenSearchPPLParser.g4

# Conflicts:
#	ppl/src/main/antlr/OpenSearchPPLParser.g4
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
…oject#4786 (opensearch-project#5011)

* transpose command implementation

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* transpose rows to columns

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added argument type missing map and hashmap

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added more validations

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added validation

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* index.md formatting fix

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* doc format

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* coderabbit review fixes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added recommended changes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added recommended changes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* trim columnName

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* per review moved to class varialble.

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* per review moved to class varialble.

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added field resolution

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* fix by removing metadata field

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* fixed explain test after removing of metadata fields in transpose result

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

---------

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
…oject#4786 (opensearch-project#5011)

* transpose command implementation

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* transpose rows to columns

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added argument type missing map and hashmap

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added more validations

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added validation

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* index.md formatting fix

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* doc format

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* coderabbit review fixes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added recommended changes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added recommended changes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* trim columnName

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* per review moved to class varialble.

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* per review moved to class varialble.

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added field resolution

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* fix by removing metadata field

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* fixed explain test after removing of metadata fields in transpose result

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

---------

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
# Conflicts:
#	integ-test/src/test/java/org/opensearch/sql/ppl/NewAddedCommandsIT.java
#	ppl/src/main/antlr/OpenSearchPPLLexer.g4
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

# Conflicts:
#	ppl/src/main/antlr/OpenSearchPPLLexer.g4
#	ppl/src/main/antlr/OpenSearchPPLParser.g4
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
…oject#4786 (opensearch-project#5011)

* transpose command implementation

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* transpose rows to columns

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added argument type missing map and hashmap

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added tests

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added more validations

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added validation

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* index.md formatting fix

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* doc format

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* coderabbit review fixes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added recommended changes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added recommended changes

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* for cross cluster failure debugging

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* trim columnName

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* per review moved to class varialble.

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* per review moved to class varialble.

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* added field resolution

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* fix by removing metadata field

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

* fixed explain test after removing of metadata fields in transpose result

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>

---------

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants