[Backport 2.x] Fix error handling while reading analyzer mapping rules

[adnapibar]

backport 6d20423 to 2.x

Signed-off-by: Rabi Panda [email protected]

Description

Add new parseWordList method that takes a parser as a parameter. It reads custom rules from settings or a file, parses and handles errors. Make error messages less verbose for rules files outside config directory.

Issues Resolved

N/A

Check List

• New functionality includes testing.
  • All tests pass
• New functionality has been documented.
  • New functionality has javadoc added
• Commits are signed per the DCO using --signoff
• Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

[github-actions]

Gradle Check (Jenkins) Run Completed with:

• RESULT: FAILURE ❌
• URL: https://build.ci.opensearch.org/job/gradle-check/6490/
• CommitID: 184b337
  Please examine the workflow log, locate, and copy-paste the failure below, then iterate to green.
  Is the failure a flaky test unrelated to your change?

[adnapibar]

This change is also going to 2.4, should we update the CHANGELOG for 2.x ?

[andrross]

This change is also going to 2.4, should we update the CHANGELOG for 2.x ?

You did the right thing here. I have a PR out now to provide some updated instructions here. Soon the skip-changelog label will work when we don't need to add a changelog entry. (for now I used autocut which has the same effect)

[dblock]

Build is failing on something new, #5121, looks like an artifact was expired/yanked from maven (?), @mch2 is on it.

[dblock]

Rebase this - the nebula problem should be fixed. Let’s make sure this passes all CI.

Trying to get a passing build in #5142.

[dblock]

Merged via #5142.