change plugin folder name to opensearch-sql

[kavithacm]

Signed-off-by: Kavitha Conjeevaram Mohan mohakavi@amazon.com

Description

Changing sql/plugin folder name to sql/opensearch-sql to fix getting unknown plugin error in OpenSearch maven build. This change provides name to sql plugin when built in opensearch

Update: changing plugin name didn’t work so adding settings.gradle file to plugin folder

Issues Resolved

closes #660

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.

[codecov-commenter]

Codecov Report

Merging #662 (9a0e6c9) into 2.1 (7cbb121) will decrease coverage by 31.91%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##                2.1     #662       +/-   ##
=============================================
- Coverage     94.67%   62.76%   -31.92%     
=============================================
  Files           280       10      -270     
  Lines          7570      658     -6912     
  Branches        558      119      -439     
=============================================
- Hits           7167      413     -6754     
+ Misses          349      192      -157     
+ Partials         54       53        -1     

Flag	Coverage Δ
query-workbench	62.76% <ø> (ø)
sql-engine	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...opensearch/planner/physical/MLCommonsOperator.java
.../expression/operator/convert/TypeCastOperator.java
...ensearch/storage/system/OpenSearchSystemIndex.java
...h/planner/logical/rule/MergeFilterAndRelation.java
...nsearch/sql/expression/window/WindowFunctions.java
...org/opensearch/sql/sql/domain/SQLQueryRequest.java
...nsearch/data/value/OpenSearchExprValueFactory.java
...ain/java/org/opensearch/sql/analysis/Analyzer.java
...rg/opensearch/sql/planner/logical/LogicalEval.java
...java/org/opensearch/sql/sql/parser/AstBuilder.java
... and 260 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7cbb121...9a0e6c9. Read the comment docs.

[joshuali925]

could you also port this PR to main

[kavithacm]

testing the build locally. once its successful i'll reopen the PR

[joshuali925]

could you also change :plugin in https://github.com/opensearch-project/sql/blob/main/DEVELOPER_GUIDE.rst#opensearch-opensearch-dashboards

[kavithacm]

could you also change :plugin in https://github.com/opensearch-project/sql/blob/main/DEVELOPER_GUIDE.rst#opensearch-opensearch-dashboards

yup, changed it

[joshuali925]

thanks

[acarbonetto]

LGTM

[prudhvigodithi]

LGTM
./gradlew properties lists opensearch-sql-plugin

allprojects: [root project 'opensearch-sql', project ':common', project ':core', project ':doctest', project ':integ-test', project ':legacy', project ':opensearch', project ':opensearch-sql-plugin', project ':ppl', project ':protocol', project ':sql']