search

Yelp / python-gearman

Gearman API - Client, worker, and admin client interfaces
http://github.com/Yelp/python-gearman/
Other
242 stars 123 forks source link

fix some bug #67

Open jiangrzh opened 9 years ago

jiangrzh commented 9 years ago

fix: 1) connection socket opt set to struct.pack('I', 1) for some 64bit OS 2) catch exception when current_request = self.handle_to_request_map[job_handle] throw KeyError

svisser commented 9 years ago

How can self.handle_to_request_map throw a KeyError for some job handles? It doesn't seem right to catch the KeyError and ignore them.

svisser commented 9 years ago

Ah: https://github.com/paiweilai1984/python-gearman/commit/eba998b2706aacd4f11ce09b7c12175803dd3a9a