• Daniel Bratell's avatar
    Revert "jumbo: stable assignment of inputs to chunks" · 2405b447
    Daniel Bratell authored
    This reverts commit 2c7a71c3.
    
    Reason for revert: Linux with chunk size 50 broke in v8 which we
    can't quickly fix, and Windows (chunk size 8) has a DWORD logging
    error that looks non-trivial. We need to fix those before landing
    this.
    
    Original change's description:
    > jumbo: stable assignment of inputs to chunks
    > 
    > Adding or removing a file from a jumbo source set causes on average
    > half of the chunks to have their inputs reallocated.
    > Derive chunk boundaries from a combination of list position and path
    > content. This is so that when a file is added or removed, only the
    > boundaries with adjacent chunks typically move.
    > For a balance between maximum chunk size and stability of partitions:
    > * Partition uniformly into the required number of chunks.
    > * Pick a "center" from each chunk by minimum hash value.
    > * Pick the boundaries between centers by maximum hash value.
    > 
    > Bug: 782863
    > Change-Id: Ie71d82b132e8145b4ed3d1141f85886a12149d5a
    > Reviewed-on: https://chromium-review.googlesource.com/1102218
    > Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
    > Reviewed-by: Dirk Pranke <dpranke@chromium.org>
    > Reviewed-by: Daniel Bratell <bratell@opera.com>
    > Commit-Queue: Dirk Pranke <dpranke@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#570623}
    
    TBR=dpranke@chromium.org,bratell@opera.com,brucedawson@chromium.org,david.barr@samsung.com
    
    Change-Id: I8c81aca34ab2876aebea6b54e847cb24be3aa27f
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 782863
    Reviewed-on: https://chromium-review.googlesource.com/1116658Reviewed-by: default avatarDaniel Bratell <bratell@opera.com>
    Commit-Queue: Daniel Bratell <bratell@opera.com>
    Cr-Commit-Position: refs/heads/master@{#570701}
    2405b447
jumbo.gni 9.47 KB