ijioio edited a comment on pull request #479:
URL: https://github.com/apache/solr/pull/479#issuecomment-1002910206
It is maybe seems a bit strange to hold two values `Response` and
`response`, but I made it for two reasons:
- I don't think that messing with the original `Response` value is a good
idea. It is a general response format used for all the core admin operations.
So there may be a chance of someone depending on its content/format.
- In case we use some field other than `response` we also need to update the
function `aggregateResults` to support two types of response types for
aggregation. In contrast, if we use the `response` field it will be compatible
with current`aggregateResults` code.
This is how response for backup status request will looks like in case of
adding `response` field:
```json
{
"responseHeader": {
"status": 0,
"QTime": 19
},
"success": {
"localhost:8983_solr": {
"responseHeader": {
"status": 0,
"QTime": 4
}
},
"localhost:8983_solr": {
"responseHeader": {
"status": 0,
"QTime": 8
}
},
"localhost:8983_solr": {
"responseHeader": {
"status": 0,
"QTime": 0
},
"STATUS": "completed",
"Response": "TaskId: 10562744829711078047 webapp=null
path=/admin/cores
params={core=techproducts_shard2_replica_n4&async=10562744829711078047&qt=/admin/cores&name=shard2&shardBackupId=md_shard2_1&action=BACKUPCORE&location=file:///users/rh/solr-8.11.1/backuppp/myBackupName/techproducts/&incremental=true&prevShardBackupId=md_shard2_0&wt=javabin&version=2}
status=0 QTime=4",
"response": [
"startTime",
"2021-12-30T06:45:55.978702Z",
"indexFileCount",
1,
"uploadedIndexFileCount",
0,
"indexSizeMB",
0.001,
"uploadedIndexFileMB",
0.001,
"shard",
"shard2",
"endTime",
"2021-12-30T06:45:55.995335Z",
"shardBackupId",
"md_shard2_1"
]
},
"localhost:8983_solr": {
"responseHeader": {
"status": 0,
"QTime": 0
},
"STATUS": "completed",
"Response": "TaskId: 10562744829713680734 webapp=null
path=/admin/cores
params={core=techproducts_shard1_replica_n1&async=10562744829713680734&qt=/admin/cores&name=shard1&shardBackupId=md_shard1_1&action=BACKUPCORE&location=file:///users/rh/solr-8.11.1/backuppp/myBackupName/techproducts/&incremental=true&prevShardBackupId=md_shard1_0&wt=javabin&version=2}
status=0 QTime=8",
"response": [
"startTime",
"2021-12-30T06:46:10.266750Z",
"indexFileCount",
21,
"uploadedIndexFileCount",
21,
"indexSizeMB",
0.006,
"uploadedIndexFileMB",
0.006,
"shard",
"shard1",
"endTime",
"2021-12-30T06:46:10.438942Z",
"shardBackupId",
"md_shard1_1"
]
}
},
"10562744829711078047": {
"responseHeader": {
"status": 0,
"QTime": 0
},
"STATUS": "completed",
"Response": "TaskId: 10562744829711078047 webapp=null
path=/admin/cores
params={core=techproducts_shard2_replica_n4&async=10562744829711078047&qt=/admin/cores&name=shard2&shardBackupId=md_shard2_1&action=BACKUPCORE&location=file:///users/rh/solr-8.11.1/backuppp/myBackupName/techproducts/&incremental=true&prevShardBackupId=md_shard2_0&wt=javabin&version=2}
status=0 QTime=4",
"response": [
"startTime",
"2021-12-30T06:45:55.978702Z",
"indexFileCount",
1,
"uploadedIndexFileCount",
0,
"indexSizeMB",
0.001,
"uploadedIndexFileMB",
0.001,
"shard",
"shard2",
"endTime",
"2021-12-30T06:45:55.995335Z",
"shardBackupId",
"md_shard2_1"
]
},
"10562744829713680734": {
"responseHeader": {
"status": 0,
"QTime": 0
},
"STATUS": "completed",
"Response": "TaskId: 10562744829713680734 webapp=null
path=/admin/cores
params={core=techproducts_shard1_replica_n1&async=10562744829713680734&qt=/admin/cores&name=shard1&shardBackupId=md_shard1_1&action=BACKUPCORE&location=file:///users/rh/solr-8.11.1/backuppp/myBackupName/techproducts/&incremental=true&prevShardBackupId=md_shard1_0&wt=javabin&version=2}
status=0 QTime=8",
"response": [
"startTime",
"2021-12-30T06:46:10.266750Z",
"indexFileCount",
21,
"uploadedIndexFileCount",
21,
"indexSizeMB",
0.006,
"uploadedIndexFileMB",
0.006,
"shard",
"shard1",
"endTime",
"2021-12-30T06:46:10.438942Z",
"shardBackupId",
"md_shard1_1"
]
},
"response": [
"collection",
"techproducts",
"numShards",
2,
"backupId",
1,
"indexVersion",
"9.0.0",
"startTime",
"2021-12-30T06:45:35.738064Z",
"indexSizeMB",
0.007
],
"status": {
"state": "completed",
"msg": "found [1056] in completed tasks"
}
}
```
Another way to solve the problem, is to put results of the operation to the
`toLog` list of the response. But this implies parsing the generated string
located in the `Response` field and extracting all the required fields from it
within the `aggregateResults` method. Looks unnecessary complex.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
