Bug 320355 - Allow write through commits on FailOverParticipant (load balancing)
Summary: Allow write through commits on FailOverParticipant (load balancing)
Status: CLOSED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: cdo.core (show other bugs)
Version: 4.0   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Eike Stepper CLA
QA Contact: Eike Stepper CLA
URL:
Whiteboard: Power to the People
Keywords: noteworthy
Depends on:
Blocks:
 
Reported: 2010-07-20 04:38 EDT by Erwin Betschart CLA
Modified: 2011-06-23 03:41 EDT (History)
0 users

See Also:

Attachments
Idea with a WriteThroughCommitContext (11.60 KB, patch)
2010-07-20 04:55 EDT, Eike Stepper CLA 		no flags Details | Diff
Patch v2 - for future reference (20.85 KB, patch)
2010-08-08 03:43 EDT, Eike Stepper CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Erwin Betschart CLA 2010-07-20 04:38:06 EDT 
Build Identifier: 4.0

A FailOverParticipant in state BACKUP should be allowed to make write through commits (like the offline clone) to the master repository.

Reproducible: Always
Comment 1 Eike Stepper CLA 2010-07-20 04:55:48 EDT 
Created attachment 174716 [details]
Idea with a WriteThroughCommitContext

I'd prefer to use the remote session of the BACKUP's repository synchronizer directly to open a transaction on the MASTER. Please test if that suits your need and we can close this as WORKSFORME.
Comment 2 Eike Stepper CLA 2010-07-20 05:11:06 EDT 
Please reopen if the approach in comment #1 is not adequate for you.
Comment 3 Erwin Betschart CLA 2010-08-06 11:16:16 EDT 
Your proposal has two main drawbacks in our case:

1. On state change of the repository the remote session gets deactivated / activated. Therefore I would have to rebuild the data structure on state change.

2. The remote session does not cache the revisions.
Comment 4 Eike Stepper CLA 2010-08-08 03:43:51 EDT 
Created attachment 176102 [details]
Patch v2 - for future reference

This new feature must be explicitely enabled in CDOServerUtil.createFailoverParticipant(), see last parameter "allowBackupCommits".
Comment 5 Eike Stepper CLA 2010-08-08 03:44:12 EDT 
Committed to HEAD
Comment 6 Eike Stepper CLA 2011-06-23 03:41:41 EDT 
Available in R20110608-1407