Project

General

Profile

Actions
Copy link

Task #4444

closed

Bug #3857: [SDKs] Python KeepClient should use different timeouts for different operations

Review 3857-python-sdk-timeouts

Added by Tim Pierce over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Tim Pierce
Target version:
2014-11-19 sprint

Description

Available for review at 3993ba4

Some notes:

  • This change replaces httplib2 with the requests module in every call where the SDK issues a request to Keep.
  • It does not explicitly implement or test multiple connection timeouts. Doing so just means issuing a timeout parameter that is a (connection_timeout, read_timeout) parameter, e.g.
client = arvados.KeepClient(api_client=api, timeout=(2, 20))
Actions
Copy link
 #1

Updated by Tim Pierce over 9 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Tom Clegg over 9 years ago

  • Description updated (diff)
  • Assigned To set to Tom Clegg
Actions
Copy link
 #3

Updated by Tom Clegg over 9 years ago

  • Status changed from New to In Progress
Actions
Copy link
 #4

Updated by Tom Clegg over 9 years ago

  • Assigned To changed from Tom Clegg to Tim Pierce
Actions
Copy link
 #5

Updated by Tim Pierce over 9 years ago

  • Assigned To changed from Tim Pierce to Tom Clegg
Actions
Copy link
 #6

Updated by Brett Smith over 9 years ago

  • Assigned To changed from Tom Clegg to Brett Smith

I understand from standup that Tom doesn't feel like he has to be the one to review this, so stealing it.

Actions
Copy link
 #7

Updated by Brett Smith over 9 years ago

  • Assigned To changed from Brett Smith to Tim Pierce
Actions
Copy link
 #8

Updated by Tim Pierce over 9 years ago

  • Assigned To changed from Tim Pierce to Brett Smith
Actions
Copy link
 #9

Updated by Brett Smith over 9 years ago

  • Assigned To changed from Brett Smith to Tim Pierce
Actions
Copy link
 #10

Updated by Tim Pierce over 9 years ago

  • Remaining (hours) set to 0.0
  • Status changed from In Progress to Resolved
Actions
Copy link

Also available in: Atom PDF