jaketf commented on issue #11151: [BEAM-9468]  Hl7v2 io
URL: https://github.com/apache/beam/pull/11151#issuecomment-604001261
 
 
   Future improvements:
   - Currently this uses the alpha API because the motivating use case of HL7v2 
-> FHIR mapping which requires the `schematizedData` field (not yet available 
in beta API).
   - For backwards compatibility the `ListHL7v2MessageIDs` always uses 
Messages.List to get message IDs and relies on the the `HL7v2IO.Read` to fetch 
the message contents. This provides a consistent flow for real-time and batch 
and provides backwards compatibility. However, in the alpha API Messages.List 
returns the actual message contents (rather than just the message IDs). This 
leads to us reading the messages contents twice for alpha HL7v2 stores. I 
suggest we address optimizing this "double fetch" in a future PR as the alpha 
API stabilizes.

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


With regards,
Apache Git Services

Reply via email to