Skip to content

Expose getJobParameters to subclasses#231

Merged
mminella merged 1 commit into
spring-projects:masterfrom
aeisenberg:master
Sep 18, 2013
Merged

Expose getJobParameters to subclasses#231
mminella merged 1 commit into
spring-projects:masterfrom
aeisenberg:master

Conversation

@aeisenberg

Copy link
Copy Markdown

Fix for
https://jira.springsource.org/browse/BATCHADM-176

There will be a corresponding pull request in batch admin to take advantage of this change.

@mminella

Copy link
Copy Markdown
Member

The change looks good to me. Going forward, if there is a Jira issue, please use a feature branch named the same as the Jira issue (as you did in the batch-admin change related to this one). Merged. Thanks!

@aeisenberg

Copy link
Copy Markdown
Author

Thanks for merging! Should we backport this to the 2.2.x branch to be compatible with batch-admin?

garyrussell pushed a commit to garyrussell/spring-batch that referenced this pull request Jan 7, 2014
See https://jira.springsource.org/browse/BATCHADM-176

Must be applied after
spring-projects#231

As part of this, I had to update the dependency on spring-batch to
3.0.0-BUILD_SNAPSHOT.

Because of this dependency change, I had to add two
methods to JobRepositorySupport.  It looks like a stub class, so I
left the two methods implemented as stubs.
mminella pushed a commit to mminella/spring-batch that referenced this pull request Jan 7, 2014
See https://jira.springsource.org/browse/BATCHADM-176

Must be applied after
spring-projects#231

As part of this, I had to update the dependency on spring-batch to
3.0.0-BUILD_SNAPSHOT.

Because of this dependency change, I had to add two
methods to JobRepositorySupport.  It looks like a stub class, so I
left the two methods implemented as stubs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants