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

Kafka read performance regression due to added header support

Details

    • Bug
    • Status: Resolved
    • P2
    • Resolution: Fixed
    • None
    • 2.29.0
    • io-java-kafka
    • None

    Description

      Support for headers in KafkaIO reads was recently added:
      https://issues.apache.org/jira/browse/BEAM-10865

      This introduced several reflection calls into the path of advancing KafkaUnboundedReader. While separately running benchmarks, I noticed this regression.

      Calls currently come from:
      ConsumerSpEL.hasHeaders -> can be cached similar to other booleans
      deserialize key and value methods -> could be avoided in cases where headers are not being examined (at a minimum can be avoided for known coders like ByteArrayDeserializer)

      Attachments

        Issue Links

          Activity

            People

              scwhittle Sam Whittle
              scwhittle Sam Whittle
              Votes:
              0 Vote for this issue
              Watchers:
              2 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 - 2h 20m
                  2h 20m