774977 - Combine "shard" and "shardsize" into a single setting.

Status: RESOLVED WONTFIX

(Cloud Services Graveyard :: Server: Sync, defect)

Description

[Ryan Kelly [:rfkelly]]

Spinning out from Bug 770159 Comment 2.

SyncStorage SQL backend currently takes "shard" and "shardsize" parameters.  It would be cleaner to combine then into a single parameter for the shard size, and if that is missing/none/zero then sharding is disabled.

Comment 1

[Ryan Kelly [:rfkelly]]

Context for contributors:

* The repo in question is at: https://github.com/mozilla-services/server-syncstorage
* The code in question is the files in this directory: https://github.com/mozilla-services/server-syncstorage/blob/master/syncstorage/storage/sql/
* You'll probably need to update some tests under: https://github.com/mozilla-services/server-syncstorage/tree/master/syncstorage/tests
* You can run the tests using `make test`, and submit via PR in github

Comment 2

[Ryan Kelly [:rfkelly]]

Seems like we're unlikely to revisit this given syncserver infra is stable.