Percent encode opensearch index name

[seankao-az]

Description

Handle special characters when generating Flint index name by percent-encoding the invalid OpenSearch index name characters, since in some cases, sql queries (DROP, ALTER) are handled over here rather than forwarded to spark.
Same fix in opensearch-spark repo: opensearch-project/opensearch-spark#215

Issues Resolved

• [FEATURE] Improve special character handling for generating Flint index name opensearch-spark#192

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.

[noCharger]

It might cause parsing issue. Let's sync up offline.

[seankao-az]

It might cause parsing issue. Let's sync up offline.

sure I get your concern. Will test if spark submit parameters can parse the backtick stuff

Edit:
actually on second thought, this PR will not introduce any parsing issue for the query in spark submit parameter. Any special characters in backticks is already supported in OpenSearch sql and ppl.
This PR is addressing the case where OS sql / ppl already accepts table names with special character, and we are trying to create an index with special characters due to that, which is forbidden by OpenSearch, hence the encoding

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.50%. Comparing base (b375a28) to head (e26053a).

Additional details and impacted files

@@            Coverage Diff            @@
##               main    #2564   +/-   ##
=========================================
  Coverage     95.50%   95.50%           
- Complexity     5113     5117    +4     
=========================================
  Files           489      489           
  Lines         14310    14318    +8     
  Branches        961      963    +2     
=========================================
+ Hits          13667    13675    +8     
  Misses          618      618           
  Partials         25       25           

Flag	Coverage Δ
sql-engine	95.50% <100.00%> (+<0.01%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dai-chen]

Thanks for the changes!