Add function that accepts StepExecution parameter to Partitioner API

[wesley-94]

Add function that accepts StepExecution parameter to Partitioner API

[pivotal-cla]

@wesley-94 Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

[injae-kim]

Closes: #4670

[pivotal-cla]

@wesley-94 Thank you for signing the Contributor License Agreement!

[injae-kim]

can you remove //? (maybe we should remove @FunctionalInterface cause we have 2 method

[wesley-94]

I modified it as you reviewed. Thank you!

[djechelon]

Thanks so much for this PR.
But this looks incomplete.

1. You have to amend the caller, which I don't see in the PR. The caller must point at the new method passing the StepExecution
2. I don't like the idea of having non-default partition(int) because what we want is both to not create regressions, but to encourage users transition to the new API. There is no other possibility than making both methods default IMO

[wesley-94]

Thanks for your answer.
But ‘No target method found’ message was found at @FunctionalInterface when using ‘default’ keyword in both of methods.