Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Gearman CPAN distribution.

Report information
The Basics
Id: 67385
Status: resolved
Priority: 0/
Queue: Gearman
People
Owner: info [...] maximka.de
Requestors: carlson [...] celldivision.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 1.130.001

History Show all quoted text
  Mon Apr 11 13:52:15 2011 carlson [...] celldivision.com - Ticket created
Subject: Gearman::Worker::work(on_complete) triggers not when complete.
Date: Mon, 11 Apr 2011 17:52:00 +0000
To: "bug-Gearman [...] rt.cpan.org" <bug-Gearman [...] rt.cpan.org>
From: Elliott Carlson <carlson [...] celldivision.com>
In Gearman::Worker 1.11 when calling the on_complete callback for the work() method it is actually triggered prior to sending any response. It would be useful to have an actual trigger at the end of all communication to acknowledge that the process has full completed. Current local workaround is to move $complete_cb->($jobhandle, $ret) if $complete_cb; at line 367 to a position after the unless statement At line 373.
  Sun Jul 31 16:44:57 2016 info [...] maximka.de - Correspondence added
Actually depending on $ret value will be called complete cb or fail cb. Referred to the suggestion execution of both fail cb and complete cb will be possible. How ever the suggestion will be reviewed.
  Sun Jul 31 16:44:57 2016 The RT System itself - Status changed from 'new' to 'open'
  Sun Jul 31 16:44:57 2016 info [...] maximka.de - Taken
  Sun Jul 31 16:44:57 2016 info [...] maximka.de - Fixed in 1.11 added
  Fri Aug 05 17:01:21 2016 info [...] maximka.de - Correspondence added
if you haven't any objections I'll set the issue status to resolved
  Fri Aug 05 17:01:27 2016 info [...] maximka.de - Fixed in 1.130.001 added
  Fri Aug 05 17:01:27 2016 info [...] maximka.de - Fixed in 1.11 deleted
  Mon Aug 15 16:36:53 2016 info [...] maximka.de - Correspondence added
done
  Mon Aug 15 16:36:53 2016 info [...] maximka.de - Status changed from 'open' to 'resolved'