Yes, I was pending to check out your patch! Thank you very much!

2014-03-03 18:57 GMT+01:00 Ben Hegarty <heg...@gmail.com>:

> just as an FYI there is a bug in the mongodb impl in the file
> MongoDbDeltaCollection.java here...
>
>  129   @Override
> 130   public WaveletDeltaRecord getDeltaByEndVersion(long version) throws
> IOException {
> 131     DBObject query = createWaveletDBQuery();
> 132
> query.put(MongoDbDeltaStoreUtil.FIELD_TRANSFORMED_RESULTINGVERSION_VERSION,
> version);
> 133
> 134     DBObject result = deltaDbCollection.findOne(query);
> 135
> 136     WaveletDeltaRecord waveletDelta = null;
> 137
> 138     if (result != null)
> 139     try {
> *140       MongoDbDeltaStoreUtil.deserializeWaveletDeltaRecord(result);*
> 141     } catch (PersistenceException e) {
> 142       throw new IOException(e);
> 143     }
> 144     return waveletDelta;
> 145   }
>
> that may cause issues, the deserialised record doesn't actually get
> returned.  Once I fixed the mongodb unit tests, I found it.
>
> Regards
> Ben
>
>
> On Mon, Mar 3, 2014 at 5:02 PM, Pablo Ojanguren <pablo...@gmail.com>
> wrote:
>
> > Great whether import/export tools are realiable. I will perform test too.
> >
> > Thanks
> >
> >
> > 2014-03-03 17:59 GMT+01:00 Yuri Z <vega...@gmail.com>:
> >
> > > I guess it might work. I ll try it out.
> > > On Mar 3, 2014 6:53 PM, "Pablo Ojanguren" <pablo...@gmail.com> wrote:
> > >
> > > > Hi,
> > > >
> > > > I wonder if we could use existing delta export/import utilities to
> > > perform
> > > > such migration. These would be the steps to perform:
> > > >
> > > > 1) Run export utility and get a full backup of deltas
> > > > 2) Change server configuration to use mongoDB for delta persistence
> > > > 3) Run import utility with previous backup files
> > > >
> > > >
> > > > Does anyone see any pitfall or risk in this approach?
> > > >
> > > > Thank you.
> > > >
> > >
> >
>
>
>
> --
> Regards
> Ben
>

Reply via email to