openstack
/
arch-wg
Code Issues Proposed changes

Update patch set 1 

Patch Set 1:

(2 comments)

Patch-set: 1
Reviewer: Gerrit User 1297 <1297@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Label: Verified=0
This commit is contained in:
Gerrit User 1297 2017-01-19 23:49:21 +00:00 committed by Gerrit Code Review
parent c793a0cd90
commit e9e4d79a9a
1 changed files with 38 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
f30b66f8e6c30f6ef0f0726f4f0b6ce8a1f13462 Normal file
View File

@ -0,0 +1,38 @@
{
"comments": [
{
"key": {
"uuid": "5a3905b3_0d6392dd",
"filename": "active/base-services.rst",
"patchSetId": 1
},
"lineNbr": 38,
"author": {
"id": 1297
},
"writtenOn": "2017-01-19T23:49:21Z",
"side": 1,
"message": "Yes please to distributed lock manager, though there has been so much recreation of what it already does that it will require loads of time to rewrite; one of the crappy side-effects of not mandating this from the start (using our brains and past experiences to predict what will happen). C\u0027est la vie.... hopefully its not to late to actually use our brains in this way and affect change.",
"revId": "f30b66f8e6c30f6ef0f0726f4f0b6ce8a1f13462",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "5a3905b3_cd96ea93",
"filename": "active/base-services.rst",
"patchSetId": 1
},
"lineNbr": 43,
"author": {
"id": 1297
},
"writtenOn": "2017-01-19T23:49:21Z",
"side": 1,
"message": "Yup.",
"revId": "f30b66f8e6c30f6ef0f0726f4f0b6ce8a1f13462",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
}
]
}