Skip to content

SQL/PPL artifact renaming and version reset #15

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
dai-chen merged 12 commits into from
Apr 23, 2021
Merged

SQL/PPL artifact renaming and version reset #15

dai-chen merged 12 commits into from
Apr 23, 2021

Conversation

@dai-chen
Copy link
Collaborator

@dai-chen dai-chen commented Apr 21, 2021 •
edited
Loading

Description

  1. Fix broken workflow by publishing OpenSearch artifact to local Maven repo. Reference: https://github.com/opensearch-project/anomaly-detection/pull/13/files
  2. Rename SQL artifact name from opendistro-sql-x.x.x.x to opensearch-sql
  3. Build SQL/PPL against OpenSearch 1.0.0-alpha1 (due to IT error)
  4. Reset plugin as well as driver version to 1.0.0
  5. Rename final install dir of ODBC to opensearch-sql-odbc (may need another PR to rename lib file and DSN name)

Issues Resolved

N/A

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.

@dai-chen dai-chen self-assigned this Apr 22, 2021
@dai-chen dai-chen marked this pull request as ready for review April 23, 2021 00:36
joshuali925
joshuali925 reviewed Apr 23, 2021
View reviewed changes
Copy link
Member

@joshuali925 joshuali925 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@dai-chen dai-chen merged commit 3e5a68b into opensearch-project:develop Apr 23, 2021
@dai-chen dai-chen deleted the fix-sql-workflow-by-local-maven-repo branch April 26, 2021 18:09
GumpacG pushed a commit that referenced this pull request Aug 17, 2023
@MitchellGale @Yury-Fridlyand
[Spotless] Applying Google Code Format for several directories #15 (#…
bb3c340 
…1975)

* Dev/sl google java format15 (#341)

* Spotless apply on 'datasources/**/*.java',
                                      'core/**/*.java',
                                      'docs/**/*.java',
                                      'doctest/**/*.java',
                                      'relase-notes/**/*.java',
                                      'spark/**/*.java'

Signed-off-by: Mitchell Gale <[email protected]>

* Added more directories to build.gradle

Signed-off-by: Mitchell Gale <[email protected]>

* spotless apply for plugin.

Signed-off-by: Mitchell Gale <[email protected]>

* Adding checkstyle ignorefailures.

Signed-off-by: Mitchell Gale <[email protected]>

---------

Signed-off-by: Mitchell Gale <[email protected]>

* added missing semi colon.

Signed-off-by: Mitchell Gale <[email protected]>

* Update build.gradle

Co-authored-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Mitchell Gale <[email protected]>

* Update build.gradle

Signed-off-by: Mitchell Gale <[email protected]>

* Update build.gradle

Signed-off-by: Mitchell Gale <[email protected]>

---------

Signed-off-by: Mitchell Gale <[email protected]>
Signed-off-by: Mitchell Gale <[email protected]>
Co-authored-by: Yury-Fridlyand <[email protected]>
songkant-aws pushed a commit to songkant-aws/sql that referenced this pull request Mar 25, 2025
@xinyual
Merge pull request opensearch-project#15 from yuancu/cleanDateTime
0021744 
Adding date / time ITs & Fix functions
sandeshkr419 added a commit to sandeshkr419/sql that referenced this pull request Dec 3, 2025
@sandeshkr419
[Fix] Incorrect Field Index Mapping in AVG to SUM/COUNT Conversion (o…
8b907c7 
…pensearch-project#15)

* avg fix field mapping

Signed-off-by: Sandesh Kumar <[email protected]>

* fix dependency resolution

Signed-off-by: Sandesh Kumar <[email protected]>

---------

Signed-off-by: Sandesh Kumar <[email protected]>
sandeshkr419 added a commit to sandeshkr419/sql that referenced this pull request Dec 3, 2025
@sandeshkr419 @github-actions
@Swiddis @manasvinibs @opensearch-ci-bot @noCharger @dai-chen @Mebsina @yuancu @opensearch-trigger-bot @ahkcs @penghuo @alexey-temnikov @RileyJergerAmazon @ykmr1224 @LantaoJin @songkant-aws @qianheng-aws @xinyual @RyanL1997 @peterzhuamazon @vinaykpud @expani @vamsimanohar @ritvibhatt @ishaoxy @mch2
Data fusion changes
6f7fd68 
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Simeon Widdis <[email protected]>
Co-authored-by: Manasvini B S <[email protected]>
Co-authored-by: opensearch-ci-bot <[email protected]>
Co-authored-by: Louis Chu <[email protected]>
Co-authored-by: Chen Dai <[email protected]>
Co-authored-by: Mebsina <[email protected]>
Co-authored-by: Yuanchun Shen <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Kai Huang <[email protected]>
Co-authored-by: Peng Huo <[email protected]>
Co-authored-by: Alexey Temnikov <[email protected]>
Co-authored-by: Riley Jerger <[email protected]>
Co-authored-by: Tomoyuki MORITA <[email protected]>
Co-authored-by: Lantao Jin <[email protected]>
Co-authored-by: Songkan Tang <[email protected]>
Co-authored-by: qianheng <[email protected]>
Co-authored-by: Simeon Widdis <[email protected]>
Co-authored-by: Xinyuan Lu <[email protected]>
Co-authored-by: Jialiang Liang <[email protected]>
Co-authored-by: Peter Zhu <[email protected]>
Co-authored-by: Vinay Krishna Pudyodu <[email protected]>
Co-authored-by: expani <[email protected]>
Co-authored-by: expani1729 <[email protected]>
Co-authored-by: Vamsi Manohar <[email protected]>
Co-authored-by: ritvibhatt <[email protected]>
Co-authored-by: Xinyu Hao <[email protected]>
Co-authored-by: Marc Handalian <[email protected]>
Co-authored-by: Marc Handalian <[email protected]>
Fix join type ambiguous issue when specify the join type with sql-like join criteria (opensearch-project#4474)
Fix issue 4441 (opensearch-project#4449)
Fix missing keywordsCanBeId (opensearch-project#4491)
Fix the bug of explicit makeNullLiteral for UDT fields (opensearch-project#4475)
Fix mapping after aggregation push down (opensearch-project#4500)
Fix percentile bug (opensearch-project#4539)
Fix JsonExtractAllFunctionIT failure (opensearch-project#4556)
Fix sort push down into agg after project already pushed (opensearch-project#4546)
Fix push down failure for min/max on derived field (opensearch-project#4572)
Fix compile issue in main (opensearch-project#4608)
Fix filter parsing failure on date fields with non-default format (opensearch-project#4616)
Fix bin nested fields issue (opensearch-project#4606)
Fix: Support Alias Fields in MIN, MAX, FIRST, LAST, and TAKE Aggregations (opensearch-project#4621)
fix rename issue (opensearch-project#4670)
Fixes for `Multisearch` and `Append` command (opensearch-project#4512)
Fix asc/desc keyword behavior for sort command (opensearch-project#4651)
Fix] Fix unexpected shift of extraction for `rex` with nested capture groups in named groups  (opensearch-project#4641)
Fix CVE-2025-48924 (opensearch-project#4665)
Fix sub-fields accessing of generated structs (opensearch-project#4683)
Fix] Incorrect Field Index Mapping in AVG to SUM/COUNT Conversion (opensearch-project#15)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joshuali925 joshuali925 joshuali925 approved these changes

+1 more reviewer

@chloe-zh chloe-zh chloe-zh approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@dai-chen dai-chen

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dai-chen @joshuali925 @chloe-zh