-
Notifications
You must be signed in to change notification settings - Fork 38
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.
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
Created release notes for 2.10 release #396
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,21 +12,11 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), | |
### Maintenance | ||
### Refactoring | ||
|
||
## [Unreleased 2.x](https://github.com/opensearch-project/geospatial/compare/2.9...2.x) | ||
## [Unreleased 2.x](https://github.com/opensearch-project/geospatial/compare/2.10...2.x) | ||
### Features | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You need to delete all logs from here when you move them to release note. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since the code freeze is not done, I am keeping these changes. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. deleted. |
||
* IP2Geo processor implementation ([#362](https://github.com/opensearch-project/geospatial/pull/362)) | ||
### Enhancements | ||
### Bug Fixes | ||
* Revert datasource state when delete fails([#382](https://github.com/opensearch-project/geospatial/pull/382)) | ||
* Update ip2geo test data url([#389](https://github.com/opensearch-project/geospatial/pull/389)) | ||
### Infrastructure | ||
* Make jacoco report to be generated faster in local ([#267](https://github.com/opensearch-project/geospatial/pull/267)) | ||
* Exclude lombok generated code from jacoco coverage report ([#268](https://github.com/opensearch-project/geospatial/pull/268)) | ||
### Documentation | ||
### Maintenance | ||
* Change package for Strings.hasText ([#314](https://github.com/opensearch-project/geospatial/pull/314)) | ||
* Refactor LifecycleComponent package path ([#377](https://github.com/opensearch-project/geospatial/pull/377)) | ||
* [Refactor] Strings utility methods to core library ([#379](https://github.com/opensearch-project/geospatial/pull/379)) | ||
* Fixed compilation errors after refactoring in core foundation classes ([#380](https://github.com/opensearch-project/geospatial/pull/380)) | ||
* Version bump for spotlss and apache commons([#400](https://github.com/opensearch-project/geospatial/pull/400)) | ||
### Refactoring |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
## Version 2.10.0.0 Release Notes | ||
|
||
Compatible with OpenSearch 2.10.0 | ||
|
||
### Features | ||
* IP2Geo processor implementation ([#362](https://github.com/opensearch-project/geospatial/pull/362)) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should have better naming for this. Impact does not show up. @heemin32 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. imo, release log is not a place where impact can be seen. |
||
|
||
### Bug Fixes | ||
* Revert datasource state when delete fails([#382](https://github.com/opensearch-project/geospatial/pull/382)) | ||
* Update ip2geo test data url([#389](https://github.com/opensearch-project/geospatial/pull/389)) | ||
|
||
### Infrastructure | ||
* Make jacoco report to be generated faster in local ([#267](https://github.com/opensearch-project/geospatial/pull/267)) | ||
* Exclude lombok generated code from jacoco coverage report ([#268](https://github.com/opensearch-project/geospatial/pull/268)) | ||
|
||
### Maintenance | ||
* Change package for Strings.hasText ([#314](https://github.com/opensearch-project/geospatial/pull/314)) | ||
* Fixed compilation errors after refactoring in core foundation classes ([#380](https://github.com/opensearch-project/geospatial/pull/380)) | ||
* Version bump for spotlss and apache commons([#400](https://github.com/opensearch-project/geospatial/pull/400)) | ||
### Refactoring | ||
* Refactor LifecycleComponent package path ([#377](https://github.com/opensearch-project/geospatial/pull/377)) | ||
* Refactor Strings utility methods to core library ([#379](https://github.com/opensearch-project/geospatial/pull/379)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the code freeze is not done, I am keeping these changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Plus 2.11 is not ready. I don't think so we should make this change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah. You are right here.. It should be
2.10...2.x.