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:///path/to/my/shared/drive/mybackup/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:///path/to/my/shared/drive/mybackup/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:///path/to/my/shared/drive/mybackup/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:///path/to/my/shared/drive/mybackup/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: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to