Skip to content

Conversation

@yuancu
Copy link

@yuancu yuancu commented May 16, 2025

Description

Support functions coalesce with Calcite, which returns the first non-null value in its parameter list.

Issues Resolved

Resolves opensearch-project#3610

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

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.

ykmr1224 and others added 30 commits June 12, 2024 15:31
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
)

* Abstract metrics to reduce dependency to legacy

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Add comment

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Fix style

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

---------

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
* Abstract FlintIndex client

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Fix log

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Fix test function name

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

---------

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
…2778)

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
)

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
* Fix SQLQueryUtils to extract multiple tables

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Improve test coverage

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

---------

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
…ect#2786)

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Manasvini B S <manasvis@amazon.com>
* Fix node issue in CI

Signed-off-by: Rupal Mahajan <maharup@amazon.com>

* fix linux ci

Signed-off-by: Rupal Mahajan <maharup@amazon.com>

---------

Signed-off-by: Rupal Mahajan <maharup@amazon.com>
…IndexDescriptors (opensearch-project#2772)

* Register system index descriptors through SystemIndexPlugin.getSystemIndexDescriptors

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Add Spark Buffer index pattern

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Upgrade to checkout v4

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Fix CI issues

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Move env

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Revert "Upgrade to checkout v4"

This reverts commit b35573b.

---------

Signed-off-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Derek Ho <dxho@amazon.com>
Co-authored-by: Derek Ho <dxho@amazon.com>
* Introduce SparkParameterComposerCollection

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Fix comments

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Fix integ test

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

---------

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Peng Huo <penghuo@gmail.com>
LantaoJin and others added 29 commits April 9, 2025 11:33
…ct#3521)

---------

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



Fix build



Fix code style

Signed-off-by: Siri Teja Gaddameedi <sirigadd@amazon.com>
Co-authored-by: Siri Teja Gaddameedi <sirigadd@amazon.com>
by adding Locale.ROOT when converting string to lowercase to avoid irregular casting of I when defaulting to Turkic languages

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* [Calcite Engine] Support alias type field

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Add standalone IT to ensure w/wo push down test cases

Signed-off-by: Heng Qian <qianheng@amazon.com>

---------

Signed-off-by: Heng Qian <qianheng@amazon.com>
… of Java-agent. (opensearch-project#3551)

* Revert "Fix build due to phasing off SecurityManager usage in favor of Java Agent (opensearch-project#3539)"

This reverts commit 4ac9295.

Signed-off-by: Pranav Reddy <pranavrd@amazon.com>

* Using java-agent gradle plugin to phase off Security Manager in favor of Java-agent.

Signed-off-by: Pranav Reddy <pranavrd@amazon.com>

---------

Signed-off-by: Pranav Reddy <pranavrd@amazon.com>
opensearch-project#3544)

* [Calcite Engine] Only enable fallback for tests that need to fall back

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Migrate all calcite remote IT and fix incorrectly ignored test

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Follow the previous action to ignore prometheus test

Signed-off-by: Heng Qian <qianheng@amazon.com>

---------

Signed-off-by: Heng Qian <qianheng@amazon.com>
---------

Signed-off-by: Heng Qian <qianheng@amazon.com>
* fix timestamp bug

Signed-off-by: xinyual <xinyual@amazon.com>

* add TODO

Signed-off-by: xinyual <xinyual@amazon.com>

* change doc

Signed-off-by: xinyual <xinyual@amazon.com>

---------

Signed-off-by: xinyual <xinyual@amazon.com>
* Fix issue 2489

Signed-off-by: Peng Huo <penghuo@gmail.com>

* Update comments

Signed-off-by: Peng Huo <penghuo@gmail.com>

* Integrate with YamlRestTest

Signed-off-by: Peng Huo <penghuo@gmail.com>

* Revert change

Signed-off-by: Peng Huo <penghuo@gmail.com>

* Fix YamlTest

Signed-off-by: Peng Huo <penghuo@gmail.com>

* Disable security.manager

Signed-off-by: Peng Huo <penghuo@gmail.com>

* Remove unused code

Signed-off-by: Peng Huo <penghuo@gmail.com>

---------

Signed-off-by: Peng Huo <penghuo@gmail.com>
* [Calcite Engine]Support metadata field

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix UT/IT

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Refine code

Signed-off-by: Heng Qian <qianheng@amazon.com>

* [Calcite engine] Fix nested subquery

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix merging

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Refine Code

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix IT

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments3

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix Anonymizer

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

---------

Signed-off-by: Heng Qian <qianheng@amazon.com>
* Support explain PPL command

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

* add cost in explain endpoint

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

* fix compile error

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

* fix IT

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

* fix pushdown IT

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

* add requireNonNull

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

* fix code style and IT

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

* fix IT

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

* Add doc and fix some bugs

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

* fix UT

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

---------

Signed-off-by: Lantao Jin <ltjin@amazon.com>
…search-project#3571)

* New added commands should throw exception when calcite disabled

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

* remove useless code

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

* address comments

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

* fix UT

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

* Add a new IT case

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

---------

Signed-off-by: Lantao Jin <ltjin@amazon.com>
* Support CASE function with Calcite

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

* add anonymizer

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

* address comment and fix varchar literal bug

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

* add doc

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

* fix doctest

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

* fix doctest

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

---------

Signed-off-by: Lantao Jin <ltjin@amazon.com>
* Support parsing documents with flattened value

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Add UT and yamlTest

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Refine code

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Refine code

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Refine code

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix UT by using LinkedHashMap to ensure order

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix UT

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix IT

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix log level for JdbcOpenSearchDataTypeConvertor

Signed-off-by: Heng Qian <qianheng@amazon.com>

---------

Signed-off-by: Heng Qian <qianheng@amazon.com>
…3590)

(cherry picked from commit 9f881fd)

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com>
Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com>
Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
…earch-project#3593)

* Add release notes on version 3.0.0.0



* Add more commits which are missing between two release notes



---------


(cherry picked from commit 98c1b6a)

Signed-off-by: Lantao Jin <ltjin@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…lcite (opensearch-project#3553)

* Revert simple_pattern window function change to recover pushdown ability

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Add SIMPLE_PATTERN patterns command support based on parse command

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Address minor comments

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Address comments part 2

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Make allowCast for pattern VARCHAR literal

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Fix spotless

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Minor ut failure fix

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Add patterns command push down cases in ExplainIT

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Fix patterns ExplainIT failure

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Rename visitWindow to visitPatterns

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Correct ExplainIT expected plan for patterns

Signed-off-by: Songkan Tang <songkant@amazon.com>

* Add patterns anonymizer test for SIMPLE_PATTERN

Signed-off-by: Songkan Tang <songkant@amazon.com>

---------

Signed-off-by: Songkan Tang <songkant@amazon.com>
…pensearch-project#2617)

Signed-off-by: Lantao Jin <ltjin@amazon.com>
Co-authored-by: Simeon Widdis <sawiddis@gmail.com>
* Support eventstats command with Calcite

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

* add doc

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

* fix IT

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

* address comments

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

* Fix conflicts

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

* address comments

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

* Support variance functions

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

* fix UT

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

* update doc

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

---------

Signed-off-by: Lantao Jin <ltjin@amazon.com>
* Implement cryptographic functions

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

* Implement cryptographic UDFs

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

* Wrap message digests with ThreadLocal to enforce thread safety

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

* Create documentation for cryptographic hash PPL functions

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

* Use calcite implementations of md5 and sha1

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

* Test field values in cryptographic UDFs

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

* Refactor: update sha2 implementations with DigestUtils

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

---------

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
…arch-project#3605)

* Add a new row count mechanism for CalciteIndexScan

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix after merging main

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Refine code

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix IT

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix IT

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix IT

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Add doc in settings.rst

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Address comments

Signed-off-by: Heng Qian <qianheng@amazon.com>

---------

Signed-off-by: Heng Qian <qianheng@amazon.com>
* refactor using timestmap

Signed-off-by: xinyual <xinyual@amazon.com>

* refactor for timestmap

Signed-off-by: xinyual <xinyual@amazon.com>

* transfer basic functions

Signed-off-by: xinyual <xinyual@amazon.com>

* add functions

Signed-off-by: xinyual <xinyual@amazon.com>

* Refactor math UDFs

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

* Refactor text UDFs

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

* Fix function name resolving when system using Turkic languages
- Problem: FunctionName.of(UPPER_CASE_I) could not be properly resolved since I is converted to dotless ı

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

* Refactor date part, date/time manipulation UDFs
- date part: hour, hour_of_day, month_of_year, month, dayofweek, day_of_week, day, dayofyear, day_of_year, dayofmonth, day_of_month, minute_of_day, minute_of_hour, minute, quarter, second, second_of_minute, microsecond
- date/time manipulation: addtime, adddate, date_sub, date_add, subtime, subdate, extract

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

* Refactor current, name, format UDFs

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

* Refactor convert tz, date/timestamp diff UDFs

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

* Refactor date / time functions with non-date/time arguments
by creating an adapt function to convert an expr function to a UDF

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

* Refactor datetime, sec_to_time, time_to_sec, sysdate, timestampadd, timediff UDFs

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

* Refactor week, nullif, ifnull, if UDFs

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

* Remove BuiltinFunctionUtils and unnecessary UDFs

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

* Patch: throw proper semantic errors for date_part-related udfs

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

* Fix bugs in datetime UDFs

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

* Refactor: move all udf to org.opensearch.sql.expression.function.udf

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

* use timestamp to replace post process

Signed-off-by: xinyual <xinyual@amazon.com>

* Add more ITs for last day & restore Prometheus settings

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

* Fix euler with UDF instead of directly returning a literal to match its logical plan

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

* Reimplment last_day UDF with expr methods

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

* Ignore tests related to parameter validaton temporarily since it's in another PR

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

* Deprecate timestamp string as data context for function properties

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

* Refactor: unify datetime udf structures

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

* Apply spotless & fix zone id acquirement

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

* Fix unixtimestamp from double when locale uses non-arabic numbers

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

* Box operands so that relection calls match signatures containing non-primitive types

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

* Use system default time zone when restoring function properties

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

* Refactor return types

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

* Use builtin case and coalesce to implement if, ifnull, nullif UDFs

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

* Replace replace UDF with builtin implementations & add docs for replace, locate

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

* Correct locate function documents

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

* Separate adddate and date_add implementations for readability

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

* Add class documentations for UDFs

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

* Create PPLReturnTypes class to store common return types that it not covered in calcite ReturnTypes class

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

* Remove udf operators with the same implementations but different names

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

* Refactor: Move type comparison earlier in Add/Sub Date function implementation

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

* Refactor: update add/sub time's implementation with adaptExprMethodWithPropertiesToUDF

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

* Refactor: Move type comparison earlier in last day function implementation

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

* Refactor: implement date function with adaptExprMethodToUDF

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

* Refactor: Move type comparison earlier in extract function implementation

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

* Refactor: Move type comparison earlier in format function implementation

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

* Refactor: Move type comparison earlier in timestampdiff function implementation

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

* Refactor: Move type comparison earlier in weekday function implementation

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

* Refactor: split add and sub date to 2 static functions

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

* Eliminate convertRelDataTypeToSqlTypeName API, replacing it with ExprType comparision

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

* Refactor: pass expr values to udf implementations instead of passing java object and expr type separately

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

* Refactor: unify types in udf to use expr types (deprecating SqlTypeName)

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

* Refactor: change return types of UDF implementations to their actual return types (instead of Object)

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

* Add a test to unix_timestamp with microseconds

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

* Minor: fix calcite datetime tests

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

* Replace sqrt udf with a call to calcite's power(x, 0.5)

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

* Replace locate udf with SqlStdOperatorTable.POSITION

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

* Fix sqrt logical plan test

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

* Fix expectedSparkSql in sqrt test

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

---------

Signed-off-by: xinyual <xinyual@amazon.com>
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
Co-authored-by: xinyual <xinyual@amazon.com>
…t#3612)

* Add UT for PredicateAnalyzer and AggregateAnalyzer

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Add copyright

Signed-off-by: Heng Qian <qianheng@amazon.com>

---------

Signed-off-by: Heng Qian <qianheng@amazon.com>
…earch-project#3621)

* Add a trimmed project before aggregate to avoid NPE in Calcite

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

* Fix IT

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

---------

Signed-off-by: Lantao Jin <ltjin@amazon.com>
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
@yuancu yuancu closed this May 16, 2025
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.

[FEATURE] Support function coalesce