Spring/spring-batch
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

RESOLVED - issue BATCH-170: Concurrent modification of StepExecution when running an asynchrounous step operation

Browse Source 
http://opensource.atlassian.com/projects/spring/browse/BATCH-170

Extract constants - no change.
This commit is contained in:
dsyer
2007-12-19 21:27:20 +00:00
parent dfa9e646fe
commit 6ad57bcb69
1 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
spring-batch-core/src/main/java/org/springframework/batch/core/domain/StepContribution.java
View File

@@ -28,6 +28,16 @@ import org.springframework.batch.repeat.RepeatContext;
*/
public class StepContribution {
/**
* Key for destruction callback in StepContext for chunk.
*/
private static final String CHUNK_EXECUTION_CONTEXT_CALLBACK = "CHUNK_EXECUTION_CONTEXT_CALLBACK";
/**
* Key for destruction callback in StepContext for step.
*/
private static final String STEP_EXECUTION_CONTEXT_CALLBACK = "STEP_EXECUTION_CONTEXT_CALLBACK";
/**
* Context attribute key for step execution. Used by monitoring and managing
* clients to inspect current step execution.
@@ -70,7 +80,7 @@ public class StepContribution {
*/
public void registerChunkContext(final RepeatContext context) {
execution.getJobExecution().registerChunkContext(context);
context.registerDestructionCallback("CHUNK_EXECUTION_CONTEXT_CALLBACK", new Runnable() {
context.registerDestructionCallback(CHUNK_EXECUTION_CONTEXT_CALLBACK, new Runnable() {
public void run() {
execution.getJobExecution().unregisterStepContext(context);
}
@@ -83,7 +93,7 @@ public class StepContribution {
*/
public void registerStepContext(final RepeatContext context) {
execution.getJobExecution().registerStepContext(context);
context.registerDestructionCallback("STEP_EXECUTION_CONTEXT_CALLBACK", new Runnable() {
context.registerDestructionCallback(STEP_EXECUTION_CONTEXT_CALLBACK, new Runnable() {
public void run() {
execution.getJobExecution().unregisterStepContext(context);
}