Huh, yes of course, did not think about that.

LGTM for merging then


On Tue, Mar 4, 2014 at 5:24 PM, Gunnar Hillert <notifications@github.com>wrote:

> In
> spring-xd-dirt/src/main/java/org/springframework/xd/dirt/plugins/job/ExpandedJobParametersConverter.java:
>
> > +	 *
> > +	 * @param jobParameters Must not be null
> > +	 * @param removeRandomParameter When {@code true}, remove the "random" parameter if exists
> > +	 * @return A JSON String representation of the {@link JobParameters}
> > +	 */
> > +	public String getJobParametersAsString(JobParameters jobParameters, boolean removeRandomParameter) {
> > +
> > +		Assert.notNull(jobParameters, "jobParameters must not be null.");
> > +
> > +		final Properties properties = this.getProperties(jobParameters);
> > +
> > +		if (removeRandomParameter) {
> > +			properties.remove(UNIQUE_JOB_PARAMETER_KEY);
> > +		}
> > +
> > +		@SuppressWarnings({ "unchecked", "rawtypes" })
>
> A limitation of Java properties unfortunately - see:
>
> http://stackoverflow.com/questions/17209260/converting-java-util-properties-to-hashmapstring-string
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/spring-projects/spring-xd/pull/589/files#r10261276>
> .
>