[java] Add maxTaskRetries option for java actor creation

[kira-lin]

Signed-off-by: Zhi Lin [email protected]

Why are these changes needed?

Related issue number

Checks

• I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
• I've run scripts/format.sh to lint the changes in this PR.
• I've included any doc changes needed for https://docs.ray.io/en/master/.
• I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
• Testing Strategy
  • Unit tests
  • Release tests
  • This PR is not tested :(

[kira-lin]

@jovany-wang I reopened this pr, sorry for the inconvenience

[jovany-wang]

Looks good to me. Thanks for the contribution.

[kfstorm]

@kira-lin Could you add a unit test case?

[kira-lin]

@kfstorm Do java tests have the utility (like ClusterUtils in python) to test object recovery? I didn't find a way to lose an object in plasma in java tests. Plus, this pr just opens the option for users, the function is implemented in ray core, and is tested in python.

[kfstorm]

@kfstorm Do java tests have the utility (like ClusterUtils in python) to test object recovery? I didn't find a way to lose an object in plasma in java tests. Plus, this pr just opens the option for users, the function is implemented in ray core, and is tested in python.

Could you try copying test_actor_restart_with_retry into Java? I don't see any special logic about deleting objects in plasma.

[kira-lin]

OK. I was thinking about testing resubmitting actor task during object reconstruction, because that's our use case.

[kira-lin]

@kfstorm ci passed

[kfstorm]

LGTM. Thanks!

[kira-lin]

@ericl @raulchen @WangTaoTheTonic @iycheng Could you please take a look and merge it if possible, appreciated