incubator-wave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pablo Ojanguren" <pablo...@gmail.com>
Subject Re: Review Request 20371: Deltas migration tool from file to mongodb store
Date Sun, 13 Jul 2014 09:43:07 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20371/
-----------------------------------------------------------

(Updated July 13, 2014, 9:43 a.m.)


Review request for wave, Ali Lown, Vicente J. Ruiz Jurado, and Yuri Zelikov.


Changes
-------

Added missing mongodb properties to the server-config.xml and server.config.example files.
No problem for the review to take so long, I am busy lately too. Thank you for your time.


Bugs: WAVE-412
    https://issues.apache.org/jira/browse/WAVE-412


Repository: wave


Description
-------

Support to live installations of Wave (and its flavors) is needed if they decide move from
file to mongodb storage for deltas. So this is a tool to move data from stores.

Additionally, MongoDB configuration has been moved from the separated property file (src/org/waveprotocol/box/server/persistence/mongodb)
to the server.config file. If no property is set, default values are taken. See CoreSettings.java
for more info.


Diffs (updated)
-----

  run-data-migration.sh PRE-CREATION 
  server-config.xml 4f1d94fa7c5250aa8bc1e5bc471b0d12b3026697 
  server.config.example f3feba04a1e8a6cfa7a08911d1ddb94eedbcf98c 
  src/org/waveprotocol/box/server/CoreSettings.java 0a6f3fd50941a112acc50031c1220f3beb1dbca9

  src/org/waveprotocol/box/server/DataMigrationTool.java PRE-CREATION 
  src/org/waveprotocol/box/server/persistence/PersistenceModule.java 14c645365604d75703dea781ad159132fcd3b45f

  src/org/waveprotocol/box/server/persistence/migration/DeltaMigrator.java PRE-CREATION 
  src/org/waveprotocol/box/server/persistence/mongodb/MongoDbProvider.java 4e290779444ac6d2a28e4573eef7fa0dd3895105

  src/org/waveprotocol/box/server/persistence/mongodb/mongodb.properties 0f95fa4842cd726aca042366531c91aa0a651d47

  test/org/waveprotocol/box/server/persistence/mongodb/AccountStoreTest.java df79f791e7462c233af0edecd86d01457dbd6227

  test/org/waveprotocol/box/server/persistence/mongodb/AttachmentStoreTest.java 46dd70b8aab90d90baa91c9643cbb4586051dc65

  test/org/waveprotocol/box/server/persistence/mongodb/CertPathStoreTest.java 62bcd74478269983a639ce50121da7703f38c336


Diff: https://reviews.apache.org/r/20371/diff/


Testing
-------

Basic manual test: all waves (12) in a local server initially stored in file where migrated
succesfully.


Thanks,

Pablo Ojanguren


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message