jaketf commented on pull request #11893:
URL: https://github.com/apache/beam/pull/11893#issuecomment-639055068


   Hmmm my latest commit was just `spotlessApply` and this entire PR's change 
(how we pass the auth token for HCAPI requests for FHIR) seems unrelated to the 
GCS object moving logic in FHIR import.
   
   It does seem like a flake. Note that only the R4 variant failed of a 
parameterized test but passed for other two FHIR verisons (DSTU2, STU3).
   The failure seems to be due to trying to move a gcs object that no longer 
exists (because another thread cleaned it up?) as evidenced by 404 on the 
source object in the stacktrace (this logic is independent of FHIR version).
   
   The relevant logic where this error is raised is in `Filesystems::copy` in 
[FhirIO.ImportFn::importBatch](https://github.com/apache/beam/blob/master/sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/healthcare/FhirIO.java#L1046).
    
   @pabloem  Have we seen other instances of this test being flaky?


----------------------------------------------------------------
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.

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


Reply via email to