Skip to content

Feature addtotals 2 #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
asifabashar merged 68 commits into from
Dec 11, 2025
Merged

Feature addtotals 2 #6

asifabashar merged 68 commits into from
Dec 11, 2025

Conversation

@asifabashar
Copy link
Owner

@asifabashar asifabashar commented Dec 11, 2025

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
added addTotals command
30388cc 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
addTotals implementation
4775ec7 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
add totals and addcoltotals implementation
778cd3d 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
add totals and addcoltotals implementation
6649a89 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
add totals and addcoltotals implementation
94f6c37 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
add totals and addcoltotals implementation
26080d0 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
add totals and addcoltotals implementation
ef09ef3 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
Update AstBuilderTest.java
20fadce 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
add totals and addcoltotals implementation
2c19101 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
addtotals and addcoltotals implementation
3b9f4dc 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
addtotals and addcoltotals implementation
b9c0370 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
added spotless
44892e3 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
added anonymizer test
dd90628 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
added anonymizer test
5843f3e 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
added anonymizer test
2a5ecd6 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
fix to doctest and inclusion to category.json
89faa8b 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
fix to doctest and inclusion to category.json
0869a1e 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
fix to unit test
7ea906b 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
spotless apply
b691ab5 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
spotless apply
aee4d8e 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
spotless apply
d2241ee 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
unit test failure as fieldname is a reserved field in addtotal
44af6b2 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
added additional integration tests and added keywordcanbeid
bc95a19 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
Merge branch 'main' into feature-addtotals-2
a38d6d3 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
debug new command in integrated cross cluster test
517d6ea 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
debug new command in integrated cross cluster test
0277b99 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
applied coderabbit review recommendations
76ed473 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
remove debug
80fd3a9 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
coderabbit recommended fixes
a5d9b75 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
remove debug
84200c2 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
asifabashar and others added 29 commits December 6, 2025 15:14
@asifabashar
fix addcolstotal test for crossclusterit
ac8f091 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
Revert "fix addcolstotal test for crossclusterit"
ea1b2dd 
This reverts commit a7b2959.

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
Merge branch 'opensearch-project:main' into default-main
718c7de
@asifabashar
fix for crosscluster it test
58c9969 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
added missing expected output for explain test
c7057c7 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
(cherry picked from commit 77ceb6e)
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
revert unneeded changes
30bf76a 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
double space nit changes
6b3e7ed 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
double space nit changes
721fdd0 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
tests for changed order test in options
f8347e5 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
to trigger git workflow to retry an unrelated failure
914588b 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
to trigger git workflow to retry an unrelated failure
a36a265 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
tests for changed order test in options
30241f0 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
Update integ-test/src/test/java/org/opensearch/sql/calcite/remote/Cal…
cc133e9 
…citeAddTotalsCommandIT.java

Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar @coderabbitai
Update integ-test/src/test/java/org/opensearch/sql/calcite/remote/Cal…
bcac087 
…citeAddTotalsCommandIT.java

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Signed-off-by: Asif Bashar <abashar@apple.com>
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
to trigger git workflow to retry an unrelated failure
c64d300 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
tests for changed order test in options
02d28f6 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
to trigger git workflow to retry an unrelated failure
01c75ed 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
tests for changed order test in options
754f477 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
to trigger git workflow to retry an unrelated failure
2b956fc 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
to trigger git workflow to retry an unrelated failure
284593f 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
index.rst updated with new commands
4becf12 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
Merge branch 'opensearch-project:main' into default-main
82adaac
@asifabashar
merged upstream changes
53d71d0 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
merged upstream changes
57927be 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
merged upstream changes
4656f0b 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
merged upstream changes
c3e1f54 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
merged upstream changes
0593c79 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
merged upstream changes
a60f5c1 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar
mo format fix for addtotals and addcoltoals
9558881 
Signed-off-by: Asif Bashar <asif.bashar@gmail.com>
@asifabashar asifabashar merged commit f45322b into feature-addtotals-3 Dec 11, 2025
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@asifabashar