New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 701670 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

[USS] Remove intermediate structure during GetAllSyncMetadata

Project Member Reported by gangwu@chromium.org, Mar 15 2017

Issue description

Add following function into MetadataBatch
bool MetadataBatch:::AddSerializedMetadata(const std::string& key, const
std::string& serialized_metadata)

Currently, populate metadata from database into MetadataBatch use too many memory since we use an intermediate structure in between the database and the MetadataBatch. Adding this function can reduce memory usage and save time during this process.
 

Comment 1 by gangwu@chromium.org, Mar 21 2017

Summary: [USS] Remove intermediate structure during GetAllSyncMetadata (was: [USS] Add MetadataBatch:::AddSerializedMetadata)
It seems we do not need to add AddSerializedMetadata, using existing API still can avoid to use intermediate structure. 

Comment 3 by gangwu@chromium.org, Mar 24 2017

Components: Services>Sync
Status: Fixed (was: Assigned)

Sign in to add a comment