Project

General

Profile

Actions

Feature #2960

open

Keep can stream GET and PUT requests

Added by Tim Pierce almost 10 years ago. Updated over 4 years ago.

Status:
New
Priority:
Normal
Assigned To:
-
Category:
Keep
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Story points:
-

Related issues

Related to Arvados - Story #9760: Remove intermediary checksum from Keep store and use cut through block forwardingRejected08/08/2016

Actions
Related to Arvados - Feature #10541: [Keep] Share buffers between overlapping/consecutive GET requests for the same blockNew

Actions
Related to Arvados Epics - Story #18342: Stream Keep data to minimize latency and memory usageNew03/01/202307/31/2023

Actions
Actions #1

Updated by Tim Pierce almost 10 years ago

  • Tracker changed from Bug to Feature
  • Subject changed from Keep streams GET and PUT requests to Keep can stream GET and PUT requests

Current implementation of Keep @ a4378cd handles both GET and PUT requests by reading an entire blob (up to 64MB) into memory, performing checksums and any other policy checks, and then delivering the block appropriately. With many concurrent clients, this can bring Keep memory usage up to 500MB and more.

A more memory-efficient solution, and lower latency from the client's point of view, would be to stream data between the client and disk as much as possible with smaller buffers. The keepclient package has some useful tools for this.

Some issues to attend to:
  • When --serialize-io is enabled, Keep must continue to read an entire block and buffer it, or else a single slow client may block other clients while they wait for disk access.
  • GET cannot issue a 402 Corruption error without hashing the block first. Possible solutions include:
    • Read the block twice through a small buffer: once to calculate the checksum, and again to stream it to the client.
    • Deliver any corruption status in the second part of a multipart/mixed message.
    • Abandon any attempt to report corruption to the client and simply log it to the data manager, on the grounds that the client is already responsible for checking for corrupt blocks anyway.
  • PUT checking for collisions, which currently relies on GetBlock, will need to be rethought to handle streaming data.
Actions #2

Updated by Tim Pierce almost 10 years ago

  • Category set to Keep
Actions #3

Updated by Ward Vandewege almost 4 years ago

  • Blocks Story #16516: Run Keepstore on local compute nodes added
Actions #4

Updated by Peter Amstutz over 2 years ago

  • Related to Story #18342: Stream Keep data to minimize latency and memory usage added
Actions #5

Updated by Peter Amstutz over 2 years ago

  • Blocks deleted (Story #16516: Run Keepstore on local compute nodes)
Actions

Also available in: Atom PDF