Skip to content

Conversation

@noCharger
Copy link
Collaborator

@noCharger noCharger commented Apr 9, 2025

Description

Add release notes on version 3.0.0.0-beta1

Related Issues

opensearch-project/opensearch-build#3747 (comment)

Check List

  • New functionality has been documented.
  • 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.

Signed-off-by: Louis Chu <clingzhi@amazon.com>
Compatible with OpenSearch and OpenSearch Dashboards Version 3.0.0.0-beta1

### Breaking Changes
* Unified OpenSearch PPL Data Type ([#3345](https://github.com/opensearch-project/sql/pull/3345))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

include breaking changes from Calcite also. #3248 (comment)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

include breaking changes from Calcite also. #3248 (comment)

Moved #3473 and #3439 which causing breaking behaviors 5445d41

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Breaking changes in v3 is here #3487

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, seems they are all PRs here rather than issues. And do we need to call out the new doc here? docs/dev/intro-v3-engine.md

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And do we need to call out the new doc here? docs/dev/intro-v3-engine.md

This PR only covers the release notes. The dev doc update can be tracked separately. However, the dev doc PR can be included in the release notes if it is merged by eod today.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already included in Documentation section.

Signed-off-by: Louis Chu <clingzhi@amazon.com>
@noCharger noCharger requested a review from penghuo April 9, 2025 21:15
Copy link
Collaborator Author

@noCharger noCharger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI failing Caused by: java.lang.ExceptionInInitializerError: Exception java.lang.NoClassDefFoundError: org/opensearch/secure_sm/SecureSM [in thread "Test worker"]

Fix opensearch-project/OpenSearch#17869
Deployment https://build.ci.opensearch.org/blue/organizations/jenkins/distribution-build-opensearch/detail/distribution-build-opensearch/10991/pipeline

Signed-off-by: Louis Chu <clingzhi@amazon.com>
Copy link
Collaborator Author

@noCharger noCharger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CI is failling due to the core change
https://github.com/opensearch-project/OpenSearch/pull/17861/files

»  Caused by: java.lang.SecurityException: Denied access to: D:\a\sql\sql\integ-test\build\testclusters\integTest-0\tmp\socket_259399671, domain ProtectionDomain  (file:/D:/a/sql/sql/integ-test/build/testclusters/integTest-0/distro/3.0.0-ARCHIVE/modules/transport-netty4/transport-netty4-client-3.0.0-beta1-SNAPSHOT.jar <no signer certificates>)

Will resolve it as a seperate issue

@noCharger noCharger merged commit 604c923 into opensearch-project:main Apr 10, 2025
20 of 22 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.

3 participants