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 and others added 30 commits October 19, 2025 06:14
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
… default duration is in seconds which will be used for duration format , added cast call for tostring with 1 argument

Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Co-authored-by: ritvibhatt <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
xinyual and others added 29 commits December 9, 2025 21:34
* fix anoymizer for search command

Signed-off-by: xinyual <[email protected]>

* pushdown match when only one equal in search command

Signed-off-by: xinyual <[email protected]>

* fix regex case

Signed-off-by: xinyual <[email protected]>

* fix UT

Signed-off-by: xinyual <[email protected]>

* fix UT

Signed-off-by: xinyual <[email protected]>

* revert match change

Signed-off-by: xinyual <[email protected]>

* fix UT by ignore the expression

Signed-off-by: xinyual <[email protected]>

* remove useless change and resolve comment

Signed-off-by: xinyual <[email protected]>

* remove useless change and resolve comment

Signed-off-by: xinyual <[email protected]>

* add test cases for metadata and timestamp identifier

Signed-off-by: xinyual <[email protected]>

* change name

Signed-off-by: xinyual <[email protected]>

---------

Signed-off-by: xinyual <[email protected]>
…pensearch-project#4843)

* Remove redundant push-down-filters derived for bucket-non-null aggregation

Signed-off-by: Heng Qian <[email protected]>

* Address comment

Signed-off-by: Heng Qian <[email protected]>

* Fix name typo

Signed-off-by: Heng Qian <[email protected]>

---------

Signed-off-by: Heng Qian <[email protected]>
* Support  eval function

Signed-off-by: Kai Huang <[email protected]>

* Updates

Signed-off-by: Kai Huang <[email protected]>

* update javadoc

Signed-off-by: Kai Huang <[email protected]>

* Update to use ARRAY_DISTINCT

Signed-off-by: Kai Huang <[email protected]>

---------

Signed-off-by: Kai Huang <[email protected]>
…oject#4837)

* Change like function to case-sensitive and add ilike for case-insensitive

Signed-off-by: Lantao Jin <[email protected]>

* change plan in clickbench

Signed-off-by: Lantao Jin <[email protected]>

* fix ut

Signed-off-by: Lantao Jin <[email protected]>

* fix UT

Signed-off-by: Lantao Jin <[email protected]>

* Update syntax: like(string, PATTERN[, case_sensitive])

Signed-off-by: Lantao Jin <[email protected]>

* fix IT

Signed-off-by: Lantao Jin <[email protected]>

* Fix IT

Signed-off-by: Lantao Jin <[email protected]>

* revert import merging

Signed-off-by: Lantao Jin <[email protected]>

* support like(string, PATTERN, bool) in v2

Signed-off-by: Lantao Jin <[email protected]>

* dedup ut

Signed-off-by: Lantao Jin <[email protected]>

---------

Signed-off-by: Lantao Jin <[email protected]>
…project#4831)

* add bucket_nullable for streamstats

Signed-off-by: Xinyu Hao <[email protected]>

* little change

Signed-off-by: Xinyu Hao <[email protected]>

* fix IT, UT and doc

Signed-off-by: Xinyu Hao <[email protected]>

* fix

Signed-off-by: Xinyu Hao <[email protected]>

* change bucketNullable getOrDefault to get

Signed-off-by: Xinyu Hao <[email protected]>

* fix error

Signed-off-by: Xinyu Hao <[email protected]>

---------

Signed-off-by: Xinyu Hao <[email protected]>
…arch-project#4784)

* Support param timefield to specify span field in timechart

Signed-off-by: Yuanchun Shen <[email protected]>

* Update doc to introduce timefield parameter

Signed-off-by: Yuanchun Shen <[email protected]>

* Update ASTBuilderTest for chart: default args are handled in rel node visitor

Signed-off-by: Yuanchun Shen <[email protected]>

* Fix ast expression builder test

Signed-off-by: Yuanchun Shen <[email protected]>

* Fix anomanyzer test

Signed-off-by: Yuanchun Shen <[email protected]>

* Support using specified timefield in per functions

Signed-off-by: Yuanchun Shen <[email protected]>

* Omit by-timestamp clause in timechart command

Signed-off-by: Yuanchun Shen <[email protected]>

* Mask timefield argument in anonymizer

Signed-off-by: Yuanchun Shen <[email protected]>

* Anonymize argument span

Signed-off-by: Yuanchun Shen <[email protected]>

---------

Signed-off-by: Yuanchun Shen <[email protected]>
…4844)

* Enable dedup pushdown

Signed-off-by: Lantao Jin <[email protected]>

* fix doctest

Signed-off-by: Lantao Jin <[email protected]>

* refactor

Signed-off-by: Lantao Jin <[email protected]>

* Disable dedup expr

Signed-off-by: Lantao Jin <[email protected]>

* fix IT

Signed-off-by: Lantao Jin <[email protected]>

* fix yaml test

Signed-off-by: Lantao Jin <[email protected]>

* add more comments in code

Signed-off-by: Lantao Jin <[email protected]>

* fix conflicts

Signed-off-by: Lantao Jin <[email protected]>

* Address comments

Signed-off-by: Lantao Jin <[email protected]>

---------

Signed-off-by: Lantao Jin <[email protected]>
* change q43

Signed-off-by: xinyual <[email protected]>

* change plan after merging latest code

Signed-off-by: xinyual <[email protected]>

---------

Signed-off-by: xinyual <[email protected]>
…roject#4877)

* Remove unnecessary filter for DataHistogram aggregation

Signed-off-by: Lantao Jin <[email protected]>

* revert changes of interval selection in DataHistogram

Signed-off-by: Lantao Jin <[email protected]>

* typo

Signed-off-by: Lantao Jin <[email protected]>

* revert one it

Signed-off-by: Lantao Jin <[email protected]>

---------

Signed-off-by: Lantao Jin <[email protected]>
* Add review instructions for CodeRabbit

Signed-off-by: Tomoyuki Morita <[email protected]>

* Disable auto review

Signed-off-by: Tomoyuki Morita <[email protected]>

---------

Signed-off-by: Tomoyuki Morita <[email protected]>
…PL (opensearch-project#4841)

* [BugFix] Fix Regex OOM when there are 10+ regex clauses

Signed-off-by: Jialiang Liang <[email protected]>

* fix unit tests

Signed-off-by: Jialiang Liang <[email protected]>

* fix tests

Signed-off-by: Jialiang Liang <[email protected]>

* fix explain tests and corresponding commands

Signed-off-by: Jialiang Liang <[email protected]>

* fix explain tests for testFilterPushDownExplain

Signed-off-by: Jialiang Liang <[email protected]>

* peng - isolate the fix logic to its own visitor class

Signed-off-by: Jialiang Liang <[email protected]>

* Directly apply Calcite CoreRules.FILTER_MERGE before VolcanoPlanner plan

Co-authored-by: Peng Huo <[email protected]>
Signed-off-by: Jialiang Liang <[email protected]>

* fix the UTs

Signed-off-by: Jialiang Liang <[email protected]>

* fix the ITs after rebase

Signed-off-by: Jialiang Liang <[email protected]>

* fix clickbench IT and more ITs

Signed-off-by: Jialiang Liang <[email protected]>

* address comments from peng

Signed-off-by: Jialiang Liang <[email protected]>

* add yaml test

Signed-off-by: Jialiang Liang <[email protected]>

---------

Signed-off-by: Jialiang Liang <[email protected]>
Co-authored-by: Peng Huo <[email protected]>
* Support composite aggregation paginating in HAVING clause

Signed-off-by: Lantao Jin <[email protected]>

* typo

Signed-off-by: Lantao Jin <[email protected]>

* refactor

Signed-off-by: Lantao Jin <[email protected]>

* refactor

Signed-off-by: Lantao Jin <[email protected]>

* Fix IT

Signed-off-by: Lantao Jin <[email protected]>

* Fix doctest and IT

Signed-off-by: Lantao Jin <[email protected]>

* secruity it

Signed-off-by: Lantao Jin <[email protected]>

* revert changes in OpenSearchIndexScan

Signed-off-by: Lantao Jin <[email protected]>

* Fix compile error

Signed-off-by: Lantao Jin <[email protected]>

* Fix v2 paginationIT

Signed-off-by: Lantao Jin <[email protected]>

* optimize request total size in compoisite agg

Signed-off-by: Lantao Jin <[email protected]>

* fix it

Signed-off-by: Lantao Jin <[email protected]>

* Refactor

Signed-off-by: Lantao Jin <[email protected]>

---------

Signed-off-by: Lantao Jin <[email protected]>
* Add basic transpiler impl

Signed-off-by: Chen Dai <[email protected]>

* Add builder

Signed-off-by: Chen Dai <[email protected]>

* Use lombok builder

Signed-off-by: Chen Dai <[email protected]>

* Modify unified query planner UT to extend new test base class

Signed-off-by: Chen Dai <[email protected]>

* Update doc with API design caveat

Signed-off-by: Chen Dai <[email protected]>

* Move opensearch spark sql dialect out of test folder

Signed-off-by: Chen Dai <[email protected]>

* Update doc and test assertion message

Signed-off-by: Chen Dai <[email protected]>

* Fix line separator and license header

Signed-off-by: Chen Dai <[email protected]>

---------

Signed-off-by: Chen Dai <[email protected]>
* Support split eval function

Signed-off-by: Kai Huang <[email protected]>

# Conflicts:
#	core/src/main/java/org/opensearch/sql/expression/function/BuiltinFunctionName.java
#	integ-test/src/test/java/org/opensearch/sql/calcite/remote/CalciteArrayFunctionIT.java
#	ppl/src/main/antlr/OpenSearchPPLLexer.g4
#	ppl/src/main/antlr/OpenSearchPPLParser.g4
#	ppl/src/test/java/org/opensearch/sql/ppl/calcite/CalcitePPLArrayFunctionTest.java
#	ppl/src/test/java/org/opensearch/sql/ppl/utils/PPLQueryDataAnonymizerTest.java

* doctest

Signed-off-by: Kai Huang <[email protected]>

* Update test cases

Signed-off-by: Kai Huang <[email protected]>

* Update to not use UDF

Signed-off-by: Kai Huang <[email protected]>

---------

Signed-off-by: Kai Huang <[email protected]>
* Add workflow for SQL CLI integration tests

Signed-off-by: Simeon Widdis <[email protected]>

* Switch SQL checkout path for CLI

Signed-off-by: Simeon Widdis <[email protected]>

* Tweak paths and mkdir for remote

Signed-off-by: Simeon Widdis <[email protected]>

* Fix filename

Signed-off-by: Simeon Widdis <[email protected]>

* Remove redundant verification

Signed-off-by: Simeon Widdis <[email protected]>

* Don't pull after clone

Signed-off-by: Simeon Widdis <[email protected]>

---------

Signed-off-by: Simeon Widdis <[email protected]>
Signed-off-by: Asif Bashar <[email protected]>
@asifabashar asifabashar merged commit ecb63ea into feature_tonumber_2 Dec 10, 2025
31 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.