Commits · a1c254acc338ab4bfe15dd6d7779dd5c5fa3de10 · Jérome Perrin / neoppod

30 Jul, 2009 12 commits

Fix method handler that drop the admin node from the manager, so it can come · a1c254ac

Grégory Wisniewski authored Jul 30, 2009

back without be rejected due to invalid server address.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1104 71dcc9de-d417-0410-9af5-da40c76e7ee4

a1c254ac

Only handle partition change notification if the partition is already filled. · 53247b42
Vincent Pelletier authored Jul 30, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1103 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
53247b42

Remove a false call to super method. · 46174b92

Grégory Wisniewski authored Jul 30, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1102 71dcc9de-d417-0410-9af5-da40c76e7ee4

46174b92

Set the partition table id after clearing it, as that process involves resetting the table id. · 8fbf715e
Vincent Pelletier authored Jul 30, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1101 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
8fbf715e
Decode error codes properly, now that they became EnumItems. · e5b973ef
Vincent Pelletier authored Jul 30, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1100 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
e5b973ef
Make error code become EnumItems to get a better string representation. · 0bfacc9e
Vincent Pelletier authored Jul 30, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1099 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
0bfacc9e

Factorise packet logging code. · 2cd1a166

Vincent Pelletier authored Jul 30, 2009

Add a more detailed logging of ERROR packets (print error code and message).


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1098 71dcc9de-d417-0410-9af5-da40c76e7ee4

2cd1a166

Don't close the connection in connectionTimeout event, because this job is done · a3e44f99

Grégory Wisniewski authored Jul 30, 2009

in event.py, now before trigger the event. It's thread-safe because the
connection is locked around the close().

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1097 71dcc9de-d417-0410-9af5-da40c76e7ee4

a3e44f99

Remove some FIXME added or already present in the TODO list. · 5dfc4fdb
Grégory Wisniewski authored Jul 30, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1096 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
5dfc4fdb

Merge two dict definition from node.py and pt.py, to protocol.py. · 3f209f29

Grégory Wisniewski authored Jul 30, 2009

Rename partition_cell_states to cell_states to match with other names.
Fix indent and unify imports.

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1095 71dcc9de-d417-0410-9af5-da40c76e7ee4

3f209f29

Rename Node.getNodeType to Node.getType for consistency with None.getState. · fbbb07ea
Grégory Wisniewski authored Jul 30, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1094 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
fbbb07ea

Rename is(Server|Client|Listening)Connection without the 'Connection' suffix, · d8a1d43d

Grégory Wisniewski authored Jul 30, 2009

because there are Connection class atttributes.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1093 71dcc9de-d417-0410-9af5-da40c76e7ee4

d8a1d43d

29 Jul, 2009 28 commits

Client's storage handler use connectionLost hook. · f532bbbc

Grégory Wisniewski authored Jul 29, 2009

Storage node register unknown storage node found in the database before loading
the content into the partition table. So an update() can now expect to known the
storage node.

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1092 71dcc9de-d417-0410-9af5-da40c76e7ee4

f532bbbc

On the way of the previous commit, handleConnectionLost is now defined and · 82c18e80

Grégory Wisniewski authored Jul 29, 2009

called from the generic handler. This allow use it in some part of client and
storage handlers.
Add some XXX...


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1091 71dcc9de-d417-0410-9af5-da40c76e7ee4

82c18e80

Clarify use of _nodeLost/_dropIt methods by: · 14eafdd4

Grégory Wisniewski authored Jul 29, 2009

- Renaming _nodeLost to _handleConnectionLost and removing the node parameter
whose the computation was done before each call to it.
- Renaming _dropIt to handleNodeLost, currently called only from the base
service handler, so the abstract definition as moved into it. In this case, a
call ensure that the node is not None.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1090 71dcc9de-d417-0410-9af5-da40c76e7ee4

14eafdd4

Remove dealWithClientFailure from the base storage event handler class and it's · a2a4aa35

Grégory Wisniewski authored Jul 29, 2009

call because the implementation do nothing. This is used only in
ClientEventHandler and so, no overloading is required.

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1089 71dcc9de-d417-0410-9af5-da40c76e7ee4

a2a4aa35

Define is(Client|Server|Listening)Connection on the BaseConnection class to · 27de0880

Grégory Wisniewski authored Jul 29, 2009

ensure any acces to those methods can be done on any connection from the event
manager without crash the applications. This way is not less consistent than the
previous where isServerConnection was defined on ClientConnection and so on...
Replace any isinstance() calls with a connection as parameter by call to the
attributes above.

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1088 71dcc9de-d417-0410-9af5-da40c76e7ee4

27de0880

A master that has just switch to the secondary state can receive a notification · 5160976a
Grégory Wisniewski authored Jul 29, 2009
```
about the cluster state.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1087 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
5160976a

Add some ideas in the TODO list. · 51576dd1

Grégory Wisniewski authored Jul 29, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1086 71dcc9de-d417-0410-9af5-da40c76e7ee4

51576dd1

Fix a typo in protocol, add unregister() method on Dispatcher for consistency · 8aed780b

Grégory Wisniewski authored Jul 29, 2009

adn remove an XXX. Unify imports for protocol, remove unused variable.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1085 71dcc9de-d417-0410-9af5-da40c76e7ee4

8aed780b

Delete some unused variables. · 6772d446

Grégory Wisniewski authored Jul 29, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1084 71dcc9de-d417-0410-9af5-da40c76e7ee4

6772d446

Remove an XXX and add a TODO entry for this issue. · 86705c29

Grégory Wisniewski authored Jul 29, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1083 71dcc9de-d417-0410-9af5-da40c76e7ee4

86705c29

Remove notifyDeadNode from client app because it's no more used. The client node · 2d1f4090

Grégory Wisniewski authored Jul 29, 2009

never send notifyNodeInformation packets to the master (and don't need to do
this), so remove the associated method handler on the master side and thus
remove an XXX.

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1082 71dcc9de-d417-0410-9af5-da40c76e7ee4

2d1f4090

Master recovery handler use PartitionTable.load() that clear the content if the · 04d6a839

Grégory Wisniewski authored Jul 29, 2009

ptid is different. 
Storage node no more use a 'ptid' attribute on the app instance but use the
getID() accessor from the PartitionTable class.
PartitionTable.clear() now also reset the ptid.
PartitionTable.load() do more checks (taken from master.recovery)


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1081 71dcc9de-d417-0410-9af5-da40c76e7ee4

04d6a839

Simplify condition: None is always lower that anything else. · 454bcdb2

Grégory Wisniewski authored Jul 29, 2009

Use max() instead of if <condition>: ...


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1080 71dcc9de-d417-0410-9af5-da40c76e7ee4

454bcdb2

Partial revert of the previous commit. A dead lock was introduced instead of · aeda6d88

Grégory Wisniewski authored Jul 29, 2009

fix a potential bug. The ptid attribute on app is still removed.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1079 71dcc9de-d417-0410-9af5-da40c76e7ee4

aeda6d88

Remove an XXX: _getPartition is no more private because it's now used by the · 9e4f6725

Grégory Wisniewski authored Jul 29, 2009

master handler to ensure that the master connection is well established. Check
of greater ptid received with incremental partition table update is now done by
the PartitionTable class.

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1078 71dcc9de-d417-0410-9af5-da40c76e7ee4

9e4f6725

Vincent Pelletier authored Jul 29, 2009

Move all neo-related code from testZODB.py to __init__.py. This reduces dramaticaly the diffs between original testZODB.py and this copy.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1077 71dcc9de-d417-0410-9af5-da40c76e7ee4

1b28f75a

Remove a never read property set on app. · 13009eb6

Vincent Pelletier authored Jul 29, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1076 71dcc9de-d417-0410-9af5-da40c76e7ee4

13009eb6

Start a master node. · df5bc052

Vincent Pelletier authored Jul 29, 2009

Use neoctl to start the cluster automatically, and enable all storage nodes.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1075 71dcc9de-d417-0410-9af5-da40c76e7ee4

df5bc052

It is possible to loose a connection before master_conn was set (for example,... · a3dadc1c

Vincent Pelletier authored Jul 29, 2009

It is possible to loose a connection before master_conn was set (for example, when reconnecting to a master being shutdown). Fix this assert to ignore that case.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1074 71dcc9de-d417-0410-9af5-da40c76e7ee4

a3dadc1c

Client node use PartitionTable.load() and ask node informations before partition · 12c970c6
Grégory Wisniewski authored Jul 29, 2009
```
table.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1073 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
12c970c6

Only implement _dropId method in BaseServiceHandler class, as we have no... · 03b71f1e

Vincent Pelletier authored Jul 29, 2009

Only implement _dropId method in BaseServiceHandler class, as we have no broadcast to do except when we are primary master.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1072 71dcc9de-d417-0410-9af5-da40c76e7ee4

03b71f1e

Raise a more specific exception when master connection was interrupted.... · 1ec19260

Vincent Pelletier authored Jul 29, 2009

Raise a more specific exception when master connection was interrupted. NotReadyException should fit this use.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1071 71dcc9de-d417-0410-9af5-da40c76e7ee4

1ec19260

Add a load() on PartitionTable() to remove some logic from handlers, add · 186ec9ae

Grégory Wisniewski authored Jul 29, 2009

docstring and comments.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1070 71dcc9de-d417-0410-9af5-da40c76e7ee4

186ec9ae

Add a TODO entry. · 15d368f0

Grégory Wisniewski authored Jul 29, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1069 71dcc9de-d417-0410-9af5-da40c76e7ee4

15d368f0

Factorize partition table loading with update(). · 86337ecc

Grégory Wisniewski authored Jul 29, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1068 71dcc9de-d417-0410-9af5-da40c76e7ee4

86337ecc

Use UUID instead of Node class instance as dictionnary keys. · 0bcad6d5

Vincent Pelletier authored Jul 29, 2009

Make display order of nodes more natural (saves from iterating in node list a second time and factorises code a bit).


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1067 71dcc9de-d417-0410-9af5-da40c76e7ee4

0bcad6d5

Unknown storage nodes have to be in temporarily down state. · 4244140f
Grégory Wisniewski authored Jul 29, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1066 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
4244140f
Add the ptid parameter to update() method of PartitionTable class for · 16e0aefd
Grégory Wisniewski authored Jul 29, 2009
```
consistency.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@1065 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
16e0aefd