Uploaded image for project: 'Beam'
  1. Beam
  2. BEAM-9856

HL7v2IO.ListHL7v2Messages should be refactored to support more parallelization

Details

    • Improvement
    • Status: Resolved
    • P3
    • Resolution: Fixed
    • None
    • 2.22.0
    • io-java-gcp
    • None

    Description

      Currently the List Messages API paginates through in a single ProcessElement Call.
      However we could get a restriction based on createTime using Messages.List filter and orderby.
       

      This is inline with the future roadmap of HL7v2 bulk export API becomes available that should allow splitting on (e.g. create time dimension). Leveraging this bulk export might be a future optimization to explore.
       

      This could take one of two forms:
      1. dyanmically splitable via splitable DoFn (sexy, beam idiomatic: make optimization the runner's problem, potentially unnecessarily complex for this use case )
      2. static splitting on some time partition e.g. finding the earliest createTime and emitting a PCollection of 1 hour partitions and paginating through each hour of data w/ in the time frame that the store spans, in a separate ProcessElement. (easy to implement but will likely have hot keys / stragglers based on "busy hours")

       

      Attachments

        Issue Links

          Activity

            People

              data-runner0 Jacob Ferriero
              data-runner0 Jacob Ferriero
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 10.5h
                  10.5h