Update patch set 3
Patch Set 3: (9 comments) Patch-set: 3
This commit is contained in:
parent
803391fa92
commit
c181646a09
|
@ -0,0 +1,27 @@
|
|||
{
|
||||
"comments": [
|
||||
{
|
||||
"key": {
|
||||
"uuid": "3f79a3b5_e58e24d0",
|
||||
"filename": "specs/configurable-introspection-data-store.rst",
|
||||
"patchSetId": 3
|
||||
},
|
||||
"lineNbr": 66,
|
||||
"author": {
|
||||
"id": 24828
|
||||
},
|
||||
"writtenOn": "2018-09-26T07:06:44Z",
|
||||
"side": 1,
|
||||
"message": "I am a bit concerned with the default suffix, it may cause name conflicts in the future, maybe we just follow the way as swift storage backend, use suffix as part of the naming, so two db fields would be enough.\n\nopinions?",
|
||||
"range": {
|
||||
"startLine": 66,
|
||||
"startChar": 13,
|
||||
"endLine": 66,
|
||||
"endChar": 41
|
||||
},
|
||||
"revId": "4ede6253a130b488b9fdc5748133bccab8952e01",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue