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/01/07 17:10:52 UTC

[GitHub] [beam] laraschmidt commented on pull request #15275: [BEAM-7386] Adding EventTimeEquiJoin

laraschmidt commented on pull request #15275:
URL: https://github.com/apache/beam/pull/15275#issuecomment-1007579994


   The other one is in but it does not work for portable runner because making it work for protable runner breaks dataflow on portable runner. :)
   
   But the changes can actually be treated as pretty independent. That one was making the function we use here not as deprecated but we can still use it in this PR even if deprecated. So this is still ready for review.


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