You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/06/03 23:34:18 UTC

[GitHub] [beam] TheNeuralBit commented on pull request #19268: [BEAM-14293] Add @yields_batches and @yields_elements

TheNeuralBit commented on PR #19268:
URL: https://github.com/apache/beam/pull/19268#issuecomment-1146452287

   Despite the inlining this PR seems to have a minor effect (0.1-0.2 microseconds/element) on map_fn_microbenchmark`, presumably due to the new branch.  Benchmarks on my desktop with Intel Xeon W-2135 CPU. 
   
   c77971053d2e:
   ```
   Fixed cost   1.6063439090633391                                                                                                                                                                                                         Per-element  7.317467689514159e-07                                                                                  
   R^2          0.9972847477152673
   ```
   
   This PR:
   ```
   Fixed cost   1.630842854329745
   Per-element  7.487253376931855e-07
   R^2          0.998137270424781
   ```


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

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

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