[jvm-packages] Fix early stopping bug

[sammynammari]

Addresses #6657

[codecov-io]

Codecov Report

Merging #6708 (ebc4290) into master (dc97b5f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6708   +/-   ##
=======================================
  Coverage   81.56%   81.56%           
=======================================
  Files          13       13           
  Lines        3760     3760           
=======================================
  Hits         3067     3067           
  Misses        693      693           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc97b5f...ebc4290. Read the comment docs.

[sammynammari]

so the only failing check is a Python/C++ code linter...which shouldn't be applicable here, yes? Is there any additional action that needs to be taken?

[trivialfis]

The linter can be ignored I will fix it.

@CodingCat Please help to review the PR.

[trivialfis]

@wbo4958 Could you please help taking a look?

[wbo4958]

LGTM

[CodingCat]

could we change the checking instead? setting default would make it consistent with other params

[wbo4958]

@sammynammari , I didn't see your response, so I made another PR for this issue #6738. I can close it if you can continue to finish yours

[CodingCat]

closing this in favor of #6738.