Project

General

Profile

Actions

Idea #21219

closed

Review Java SDK PR #220

Added by Brett Smith 5 months ago. Updated 3 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

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 5 months ago

  • Assigned To set to Brett Smith
Actions #3

Updated by Brett Smith 5 months 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 5 months 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 5 months ago

  • Description updated (diff)
Actions #6

Updated by Brett Smith 5 months ago

  • Description updated (diff)
Actions #7

Updated by Brett Smith 5 months ago

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

Updated by Brett Smith 4 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 3 months ago

  • Release set to 69
Actions

Also available in: Atom PDF