Project

General

Profile

Actions

Idea #21219

closed

Review Java SDK PR #220

Added by Brett Smith about 1 year ago. Updated 11 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
SDKs
Start date:
Due date:
Story points:
-
Release relationship:
Auto

Description

https://github.com/arvados/arvados/pull/220 - This branch by Krzysztof Majewski exposes configuration settings for the connectTimeout, readTimeout, and writeTimeout used by the Java SDK client. The default for all values is 60000 (1 minute), which matches the hardcoded value used in previous versions.


Related issues 1 (0 open1 closed)

Related to Arvados - Support #21350: Merge java sdk PR to 2.7.1 and build a 2.7.1 of java sdkResolvedPeter AmstutzActions
Actions #2

Updated by Peter Amstutz about 1 year ago

  • Assigned To set to Brett Smith
Actions #3

Updated by Brett Smith about 1 year ago

  • Status changed from New to In Progress

The PR has a couple merge commits without the DCO line. 21219-java-sdk-pr-220 @ 2b9c5e2aa6e7ac19237a4cf27572a0fc99909283 has the key commit cherry-picked on top of recent main. developer-run-tests: #3940

Actions #4

Updated by Brett Smith about 1 year ago

Brett Smith wrote in #note-3:

The PR has a couple merge commits without the DCO line. 21219-java-sdk-pr-220 @ 2b9c5e2aa6e7ac19237a4cf27572a0fc99909283 has the key commit cherry-picked on top of recent main. developer-run-tests: #3940

Failed on #21258, which is a known issue affecting a lot of runs. I'm calling this good, the patch looks exactly as you would expect for what it's implementing.

Actions #5

Updated by Brett Smith about 1 year ago

  • Description updated (diff)
Actions #6

Updated by Brett Smith about 1 year ago

  • Description updated (diff)
Actions #7

Updated by Brett Smith about 1 year ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Resolved
Actions #8

Updated by Brett Smith 12 months ago

  • Related to Support #21350: Merge java sdk PR to 2.7.1 and build a 2.7.1 of java sdk added
Actions #9

Updated by Peter Amstutz 11 months ago

  • Release set to 69
Actions

Also available in: Atom PDF