Add Backward Compatibility and Validation checks to ModelGraveyard XContent Bugfix

[naveentatikonda]

Add Backward Compatibility and Validation checks to ModelGraveyard XContent Bugfix

Signed-off-by: Naveen Tatikonda [email protected]

Description

• Update ModelGraveyard fromXContent logic to add backward compatibility to those domains that have parsed the old toXContent logic.
• Add more validations in fromXContent logic.
• Add more test cases to validate them

Check List

• New functionality includes testing.
  • All tests pass
• Commits are signed as 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 #692 (e902958) into main (c412c8a) will increase coverage by 0.02%.
The diff coverage is 88.88%.

@@             Coverage Diff              @@
##               main     #692      +/-   ##
============================================
+ Coverage     84.46%   84.49%   +0.02%     
- Complexity     1055     1059       +4     
============================================
  Files           149      149              
  Lines          4307     4320      +13     
  Branches        385      389       +4     
============================================
+ Hits           3638     3650      +12     
- Misses          489      490       +1     
  Partials        180      180              

Impacted Files	Coverage Δ
...ava/org/opensearch/knn/indices/ModelGraveyard.java	90.58% <88.88%> (+0.31%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[martin-gaievski]

Looks good, thx

[navneet1v]

Approving the changes, as I can see some cosmetic comments on the code.