Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

file based export resulting in multiple files is not reported correctly by $bulkdata-status #2673

Closed
kmbarton423 opened this issue Aug 9, 2021 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@kmbarton423
Copy link
Contributor

https://localhost:9443/fhir-server/api/v4/Patient/$export?_type=Coverage

Above export resulted in six ndjson files as expected yet $bulkdata-status only reports one and the count is wrong.

{
"transactionTime": "2021-08-09T19:37:42.643Z",
"request": "https://localhost:9443/fhir-server/api/v4/Patient/$export?_type=Coverage",
"requiresAccessToken": false,
"output": [
{
"type": "Coverage",
"url": "75ngSFE9UmyaZUFz4CHuBIuHC6CY5N1yjrMRSTLTeQ0/Coverage_1.ndjson",
"count": 138
}
]
}

@kmbarton423 kmbarton423 added the bug Something isn't working label Aug 9, 2021
@lmsurpre lmsurpre added this to the Sprint 2021-11 milestone Aug 9, 2021
lmsurpre added a commit that referenced this issue Aug 10, 2021
…ort (#2672)

* Set the resourceTypeSummary at the end of each file export

Now the FileProvider follows the same pattern as the S3Provider...we
update the resourceTypeSummary at the end of each "currentUpload".

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>

* Refactored common setResourceTypeSummary code into shared util

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
@kmbarton423
Copy link
Contributor Author

Fix looks good.

{
"transactionTime": "2021-08-10T20:35:08.401Z",
"request": "https://localhost:9443/fhir-server/api/v4/Patient/$export?_type=Coverage",
"requiresAccessToken": false,
"output": [
{
"type": "Coverage",
"url": "cGXlR_G2ttEpRLq8e5fh3si7lg3o_fDasiiSc3J2C2M/Coverage_1.ndjson",
"count": 415
},
{
"type": "Coverage",
"url": "cGXlR_G2ttEpRLq8e5fh3si7lg3o_fDasiiSc3J2C2M/Coverage_2.ndjson",
"count": 415
},
{
"type": "Coverage",
"url": "cGXlR_G2ttEpRLq8e5fh3si7lg3o_fDasiiSc3J2C2M/Coverage_3.ndjson",
"count": 413
},
{
"type": "Coverage",
"url": "cGXlR_G2ttEpRLq8e5fh3si7lg3o_fDasiiSc3J2C2M/Coverage_4.ndjson",
"count": 405
},
{
"type": "Coverage",
"url": "cGXlR_G2ttEpRLq8e5fh3si7lg3o_fDasiiSc3J2C2M/Coverage_5.ndjson",
"count": 401
},
{
"type": "Coverage",
"url": "cGXlR_G2ttEpRLq8e5fh3si7lg3o_fDasiiSc3J2C2M/Coverage_6.ndjson",
"count": 80
}
]
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants