search

lucidsoftware / piezo

Piezo is a set of tools for operating a quartz scheduling cluster.
Apache License 2.0
53 stars 28 forks source link

Edit job's data map seems buggy #38

Closed pablote closed 8 years ago

pablote commented 8 years ago

When adding several data map variables sometimes only one gets added, sometimes none. What seems to work is adding one at the time.

Check out this vid: https://dl.dropboxusercontent.com/u/24172839/piezo-jobdatamap.mp4

thanks

pablote commented 8 years ago

I think I can see the reason this is happening, params posted seem to be stepping over each other. This is a sample post of two params as seen on Chrome network tab:

group:mq
name:pop-and-log
class:com.cinchcast.telephony.tasks.job.quartz.MqLoggingJob
description:Logs MQ messages
durable:true
job-data-map[0].key:mqHost
job-data-map[0].value:example.com
job-data-map[0].key:mqPort
job-data-map[0].value:1234
lucidchart commented 8 years ago

Thanks. I'd love a pull request to fix it.

Or, we may have time for someone to work on a bunch of the backlog items this summer.

On Wed, Mar 9, 2016 at 11:16 AM, Pablo Compagni notifications@github.com wrote:

I think I can see the reason this is happening, params posted seem to be stepping over each other. This is a sample post of two params as seen on Chrome network tab:

group:mq name:pop-and-log class:com.cinchcast.telephony.tasks.job.quartz.MqLoggingJob description:Logs MQ messages durable:true job-data-map[0].key:mqHost job-data-map[0].value:example.com job-data-map[0].key:mqPort job-data-map[0].value:1234

— Reply to this email directly or view it on GitHub https://github.com/lucidsoftware/piezo/issues/38#issuecomment-194433552.

margkam commented 8 years ago

Derrick thinks this is fixed. Please reopen if this is a problem.