HomePhabricator

Increase ReadingLists item limit to 5k
74b5891ee142Unpublished

Unpublished Commit · Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.
This commit no longer exists in the repository. It may have been part of a branch which was deleted.This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

Increase ReadingLists item limit to 5k

Apparently on Android having larger than 1k lists is not that unusual.

This should not have much performance effects as queries are paged and
page sizes do not change; the only place where it matters is the
COUNT(*) query that's validating the limit on insertion, and that's
covered by an index.

Bug: T186296
Change-Id: Ia3c1e631d06cc90fdbd52b121a67f68050116f88

Details

Provenance
TgrAuthored on Feb 11 2018, 8:29 PM
ChangeId
Ia3c1e631d06cc90fdbd52b121a67f68050116f88

Commit No Longer Exists

This commit no longer exists in the repository.