Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ReallyRunning event misinterpreted after resubmit from backlogged WMS (#92612) #8

Open
sustr4 opened this issue May 3, 2013 · 0 comments
Assignees
Labels

Comments

@sustr4
Copy link
Contributor

sustr4 commented May 3, 2013

This is a copy of original issue https://savannah.cern.ch/bugs/?92612 by Zdenek Salvet

ReallyRunning event is not handled correctly after deep resubmit
when it arrives earlier then Resubmission event from WMS
(WMS is not able to send events to LB server quickly enough).
"ReallyRunning on bad branch" is logged instead.

Zdenek Salvet


How about this? The idea is to perform the same action as for the RESUBMISSION event when the new event is apparently on newer branch (ie. the control went through WM once more), but the RESUBMISSION event itself is missing. I will not pretend I grasp the code fully, so please double check.

diff -u -r1.34 process_event.c
--- process_event.c 14 Dec 2011 10:44:18 -0000 1.34
+++ process_event.c 11 Apr 2012 12:16:30 -0000
@@ -323,8 +323,12 @@
fine_res = RET_GOODBRANCH;
}
else {

  • res = RET_LATE;
  • fine_res = RET_BADBRANCH;
  • if(before_deep_resubmission(js->branch_tag_seqcode, e->any.seqcode)) {
  • reset_branch(js, e);
  • } else {
  • res = RET_LATE;
  • fine_res = RET_BADBRANCH;
  • }
    }
    }
    else if ((js->last_seqcode != NULL) &&

Michal Vocu

@ghost ghost assigned mvocu May 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants