Skip to content

Conversation

@LantaoJin
Copy link
Member

@LantaoJin LantaoJin commented Mar 14, 2025

Description

  1. Revert aggregation result ordering of stats-by introduced by Keep aggregation in Calcite consistent with current PPL behavior #3405.
  2. Fix the CR-LF issue

Related Issues

Resolves #3426 and #3425

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.

Signed-off-by: Lantao Jin <ltjin@amazon.com>
Signed-off-by: Lantao Jin <ltjin@amazon.com>
@LantaoJin LantaoJin added the calcite calcite migration releated label Mar 14, 2025
@LantaoJin LantaoJin changed the title revert result ordering of stats-by Revert result ordering of stats-by Mar 14, 2025
Signed-off-by: Lantao Jin <ltjin@amazon.com>
@LantaoJin LantaoJin merged commit c82a2ed into opensearch-project:feature/calcite-engine Mar 14, 2025
12 of 13 checks passed
penghuo pushed a commit that referenced this pull request Jun 16, 2025
* revert result ordering of stats-by

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

* Fix CRLF issue

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

* only check spark sql

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

---------

Signed-off-by: Lantao Jin <ltjin@amazon.com>
Signed-off-by: xinyual <xinyual@amazon.com>
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.

2 participants