Skip to content
Browse files

GetUpdateJob was returning incorrect value

  • Loading branch information...
1 parent 376ada0 commit 0580ca24d6c8915964edf711f60729f10dacfb9a @dzerbino dzerbino committed Mar 7, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 batchSystems/gridengine.py
View
2 batchSystems/gridengine.py
@@ -255,7 +255,7 @@ def getUpdatedJob(self, maxWait):
i = None
@dentearl
Collaborator
dentearl added a note Mar 7, 2012

Shouldn't this initialization be
i = (None, None, None)
Since you're now expecting this function to return a triple? See the call made by getUpdatedJobs().

@dzerbino
Collaborator
dzerbino added a note Mar 7, 2012
@dzerbino
Collaborator
dzerbino added a note Mar 7, 2012
@dzerbino
Collaborator
dzerbino added a note Mar 7, 2012
@dentearl
Collaborator
dentearl added a note Mar 7, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
try:
(job, task, retcode) = self.updatedJobsQueue.get(timeout=maxWait)
- i = retcode
+ i = (self.jobIDs[(job, task)], retcode)
self.currentjobs -= set([self.jobIDs[(job, task)]])
except Empty:
pass

0 comments on commit 0580ca2

Please sign in to comment.
Something went wrong with that request. Please try again.