And it looks like some of your changes may have not merged correctly. I’m getting compile errors like:
The method close() is undefined for the type Transaction This shouldn’t have come from our merge. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 23, 2013, at 3:52 PM, Darren Shepherd <darren.s.sheph...@gmail.com> wrote: > Chris, Edison, > > You guys just committed 'Support Revert VM Disk from Snapshot.' At > the same time I was merging both my txn-refactor and > spring-modularization branches. They are really tricky merges and > each time I have to rebase it takes awhile to figure out. Anyhow, > your change + my changes breaks master. So I quickly rebased rb14823 > and committed to master. rb14823 is the patch that makes the Storage > Strategies work with my spring work plus clean up some things. > Additionally I found out you can't inject List<SnapshotStrategy> to > the Snapshot object, so we really have to go with my change to > centralize the ownership of the strategies to a single class. > > Can you please pull master and revalidate that I didn't break > anything, if its not too much of a pain. > > Thanks, > Darren
