add remark field to NodeData and TaskInfo

[XiaochenNan]

Description

add remark field to NodeData and TaskInfo

Issues Resolved

[#6552]

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/12027/
• CommitID: 7966cfc
  Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
  Is the failure a flaky test unrelated to your change?

[github-actions]

Gradle Check (Jenkins) Run Completed with:

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

[github-actions]

Gradle Check (Jenkins) Run Completed with:

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

[github-actions]

Gradle Check (Jenkins) Run Completed with:

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

[github-actions]

Gradle Check (Jenkins) Run Completed with:

• RESULT: UNSTABLE ❕
• TEST FAILURES:

      1 org.opensearch.cluster.routing.WeightedRoutingIT.testClusterHealthResponseWithEnsureNodeWeighedInParam

• URL: https://build.ci.opensearch.org/job/gradle-check/12078/
• CommitID: 06204ea
  Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

[saratvemulapalli]

Thanks @XiaochenNan for contributing a change.

Trying to understand, what do intend to use this field for which attributes is unable to serve?
Also it would be great if you could add tests.

[XiaochenNan]

@saratvemulapalli
just want to get the remark info in first level of data, then user can get it without analyze the key in attributes

[saratvemulapalli]

@saratvemulapalli just want to get the remark info in first level of data, then user can get it without analyze the key in attributes

Thanks @XiaochenNan. I am not really seeing value of adding a new field but I would like to hear from other maintainers @anasalkouz @reta @andrross @dblock @nknize

[andrross]

@XiaochenNan Can you explain the problem this solution is trying to solve? "user may need set more additional infomation in NodeData and TaskInfo data" from the linked issue isn't nearly detailed enough. What is the problem that leads to a user needing addition information in NodeData and TaskInfo? Once the problem is clear we can start evaluating solutions.

[opensearch-trigger-bot]

This PR is stalled because it has been open for 30 days with no activity. Remove stalled label or comment or this will be closed in 7 days.

[opensearch-trigger-bot]

This PR was closed because it has been stalled for 7 days with no activity.

[kotwanikunal]

Apologies. This PR was auto closed without reaching a resolution from the maintainers.
Re-opening to move it forward.
Thanks for your contributions to OpenSearch!

[codecov]

Codecov Report

Merging #6553 (06204ea) into main (5afc24f) will decrease coverage by 0.08%.
Report is 888 commits behind head on main.
The diff coverage is 9.52%.

@@             Coverage Diff              @@
##               main    #6553      +/-   ##
============================================
- Coverage     70.76%   70.68%   -0.08%     
+ Complexity    59122    59087      -35     
============================================
  Files          4804     4804              
  Lines        283096   283116      +20     
  Branches      40811    40813       +2     
============================================
- Hits         200326   200131     -195     
- Misses        66388    66555     +167     
- Partials      16382    16430      +48     

Files Changed	Coverage Δ
...ain/java/org/opensearch/client/tasks/NodeData.java	44.11% <6.66%> (-9.59%)	⬇️
...ain/java/org/opensearch/client/tasks/TaskInfo.java	59.78% <16.66%> (-3.01%)	⬇️

... and 509 files with indirect coverage changes

[github-actions]

Compatibility status:

Checks if related components are compatible with change 06204ea

Incompatible components

Skipped components

Compatible components

[github-actions]

Gradle Check (Jenkins) Run Completed with:

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

[ashking94]

@XiaochenNan - Is this being worked upon? Please reach out to the maintainers and other active members to reach conclusion or reviews.

[ticheng-aws]

Hi @XiaochenNan, do we have any updates on this PR?

[andrross]

I'm closing this due to no response to the questions above. @XiaochenNan feel free to respond to the feedback and reopen if appropriate. Thanks!