Update patch set 1

Patch Set 1:

(4 comments)

Patch-set: 1
CC: Gerrit User 1131 <1131@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Attention: {"person_ident":"Gerrit User 9642 \u003c9642@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_1131\u003e replied on the change"}
This commit is contained in:
Gerrit User 1131 2022-09-09 21:01:57 +00:00 committed by Gerrit Code Review
parent 7f58afd3f8
commit 5395beaae0
1 changed files with 84 additions and 0 deletions

View File

@ -0,0 +1,84 @@
{
"comments": [
{
"unresolved": false,
"key": {
"uuid": "bf2582cc_f87d8641",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 1
},
"lineNbr": 0,
"author": {
"id": 1131
},
"writtenOn": "2022-09-09T21:01:57Z",
"side": 1,
"message": "Thanks for the spec, some comments.",
"revId": "89528429f91ae1a9667a9a53d7c64bc4b58f9608",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "1577d820_740237b9",
"filename": "specs/2023.1/metadata-rate-limit.rst",
"patchSetId": 1
},
"lineNbr": 15,
"author": {
"id": 1131
},
"writtenOn": "2022-09-09T21:01:57Z",
"side": 1,
"message": "Although I understand what you mean, should probably use a better word or words, like \"overwhelming\" ? or some DoS word ?",
"range": {
"startLine": 15,
"startChar": 31,
"endLine": 15,
"endChar": 40
},
"revId": "89528429f91ae1a9667a9a53d7c64bc4b58f9608",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "cb615a6b_af43deeb",
"filename": "specs/2023.1/metadata-rate-limit.rst",
"patchSetId": 1
},
"lineNbr": 21,
"author": {
"id": 1131
},
"writtenOn": "2022-09-09T21:01:57Z",
"side": 1,
"message": "s/Platform",
"range": {
"startLine": 21,
"startChar": 0,
"endLine": 21,
"endChar": 9
},
"revId": "89528429f91ae1a9667a9a53d7c64bc4b58f9608",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "c21d141e_b4932ac0",
"filename": "specs/2023.1/metadata-rate-limit.rst",
"patchSetId": 1
},
"lineNbr": 47,
"author": {
"id": 1131
},
"writtenOn": "2022-09-09T21:01:57Z",
"side": 1,
"message": "Can you take a look at one of the zed specs and add some sections here that seem to be missing? Like Impact sections and Implementation/assignee, Testing, etc?",
"revId": "89528429f91ae1a9667a9a53d7c64bc4b58f9608",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}