codecov[bot] edited a comment on pull request #16449:
URL: https://github.com/apache/beam/pull/16449#issuecomment-1007058009
--
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.
laraschmidt commented on pull request #15275:
URL: https://github.com/apache/beam/pull/15275#issuecomment-1007579994
--
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 unsu
ihji commented on pull request #16433:
URL: https://github.com/apache/beam/pull/16433#issuecomment-1007820932
@danthev Thanks for adding tests. Can you please fix formatting errors?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
TheNeuralBit commented on a change in pull request #16256:
URL: https://github.com/apache/beam/pull/16256#discussion_r780528895
##
File path: sdks/python/apache_beam/dataframe/frames_test.py
##
@@ -1245,6 +1245,39 @@ def test_idxmax(self):
self._run_test(lambda s2: s2.idxm
suztomo commented on pull request #16441:
URL: https://github.com/apache/beam/pull/16441#issuecomment-1007446146
Java_PVR_Flink_Docker (not my interest) failed and other test succeeded.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
codecov[bot] edited a comment on pull request #16460:
URL: https://github.com/apache/beam/pull/16460#issuecomment-1007865663
--
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.
codecov[bot] commented on pull request #16456:
URL: https://github.com/apache/beam/pull/16456#issuecomment-1007619975
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16456?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+
codecov[bot] edited a comment on pull request #16250:
URL: https://github.com/apache/beam/pull/16250#issuecomment-995255415
--
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.
vitaly-ivanov commented on pull request #16451:
URL: https://github.com/apache/beam/pull/16451#issuecomment-1007431138
--
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 un
RustedBones commented on pull request #16271:
URL: https://github.com/apache/beam/pull/16271#issuecomment-1007453667
There are no breaking changes to the `AvroCoder` directly.
I'd like to add the possibility to create coders using datum factories.
Users could customize (de)serialization
codecov[bot] commented on pull request #16460:
URL: https://github.com/apache/beam/pull/16460#issuecomment-1007865663
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16460?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+
codecov[bot] commented on pull request #16461:
URL: https://github.com/apache/beam/pull/16461#issuecomment-1007868249
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16461?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+
KevinGG commented on a change in pull request #16449:
URL: https://github.com/apache/beam/pull/16449#discussion_r780473158
##
File path: sdks/python/apache_beam/internal/metrics/metric.py
##
@@ -262,4 +262,8 @@ def
bigtable_error_code_to_grpc_status_string(grpc_status_code):
y1chi commented on pull request #16439:
URL: https://github.com/apache/beam/pull/16439#issuecomment-1007662186
Run Java PreCommit
--
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 com
aromanenko-dev commented on pull request #16454:
URL: https://github.com/apache/beam/pull/16454#issuecomment-1007591803
--
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 u
codecov[bot] edited a comment on pull request #16154:
URL: https://github.com/apache/beam/pull/16154#issuecomment-988168868
--
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.
codecov[bot] edited a comment on pull request #16454:
URL: https://github.com/apache/beam/pull/16454#issuecomment-1007607190
--
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.
codecov[bot] commented on pull request #16458:
URL: https://github.com/apache/beam/pull/16458#issuecomment-1007741254
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16458?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+
codecov[bot] commented on pull request #16454:
URL: https://github.com/apache/beam/pull/16454#issuecomment-1007607190
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16454?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+
chamikaramj commented on pull request #16453:
URL: https://github.com/apache/beam/pull/16453#issuecomment-1007564804
LGTM. Thanks.
--
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 co
codecov[bot] edited a comment on pull request #16433:
URL: https://github.com/apache/beam/pull/16433#issuecomment-1007705298
--
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.
codecov[bot] edited a comment on pull request #16461:
URL: https://github.com/apache/beam/pull/16461#issuecomment-1007868249
--
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.
codecov[bot] commented on pull request #16462:
URL: https://github.com/apache/beam/pull/16462#issuecomment-1007872773
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16462?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+
lukecwik commented on pull request #15675:
URL: https://github.com/apache/beam/pull/15675#issuecomment-1007801930
--add-opens should be done for all modules that exist on the classpath and
only access to jamm should be provided. If jamm doesn't support java 9 modules
yet then we can modify
codecov[bot] edited a comment on pull request #16462:
URL: https://github.com/apache/beam/pull/16462#issuecomment-1007872773
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16462?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+
KevinGG commented on pull request #16449:
URL: https://github.com/apache/beam/pull/16449#issuecomment-1007742888
Additionally filed https://github.com/googleapis/python-bigtable/issues/485
and included a code snippet that verified the bug did exist and None types were
returned in the analy
KhaninArtur commented on pull request #16361:
URL: https://github.com/apache/beam/pull/16361#issuecomment-1007302996
@aijamalnk @aaltay our team is on holiday until the 10th of January, that is
why there is no recent activity in this PR.
This and some other PRs are at the internal re
lukecwik merged pull request #16447:
URL: https://github.com/apache/beam/pull/16447
--
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-unsubsc
codecov[bot] commented on pull request #16433:
URL: https://github.com/apache/beam/pull/16433#issuecomment-1007705298
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16433?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+
danthev commented on pull request #16433:
URL: https://github.com/apache/beam/pull/16433#issuecomment-1007705849
--
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 unsubscr
ryanthompson591 commented on a change in pull request #16250:
URL: https://github.com/apache/beam/pull/16250#discussion_r780511727
##
File path: sdks/python/apache_beam/internal/cloudpickle_pickler.py
##
@@ -99,6 +103,14 @@ def _create_absl_flags():
return flags.FLAGS
+d
ihji commented on pull request #16462:
URL: https://github.com/apache/beam/pull/16462#issuecomment-1007870251
R: @emilymye
CC: @danthev
--
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 s
codecov[bot] edited a comment on pull request #16458:
URL: https://github.com/apache/beam/pull/16458#issuecomment-1007741254
--
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.
benWize commented on pull request #16154:
URL: https://github.com/apache/beam/pull/16154#issuecomment-1007717399
--
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 unsubscr
aijamalnk commented on pull request #16361:
URL: https://github.com/apache/beam/pull/16361#issuecomment-1007696492
@KhaninArtur Thanks Artur :) and apologies for bothering during well earned
holiday break.
--
This is an automated message from the Apache Git Service.
To respond to the mes
lukecwik commented on pull request #16458:
URL: https://github.com/apache/beam/pull/16458#issuecomment-1007729456
--
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 unsubsc
kileys commented on pull request #15549:
URL: https://github.com/apache/beam/pull/15549#issuecomment-1007782447
R: @pabloem
Thanks Pablo!
--
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
ihji commented on pull request #16461:
URL: https://github.com/apache/beam/pull/16461#issuecomment-1007864703
--
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
reuvenlax commented on pull request #15275:
URL: https://github.com/apache/beam/pull/15275#issuecomment-1007610045
--
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 unsubs
aromanenko-dev commented on pull request #16456:
URL: https://github.com/apache/beam/pull/16456#issuecomment-1007597608
--
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 u
laraschmidt edited a comment on pull request #15275:
URL: https://github.com/apache/beam/pull/15275#issuecomment-1007632856
That was for an earlier iteration. The solution we ended up going with only
outputs from processElement and only uses timers to hold the watermark. So we
didn't end u
codecov[bot] edited a comment on pull request #15675:
URL: https://github.com/apache/beam/pull/15675#issuecomment-937364403
--
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.
chamikaramj commented on pull request #16461:
URL: https://github.com/apache/beam/pull/16461#issuecomment-1007869089
Thanks.
Can you also run a Google code import and make sure nothing fails ?
cc: @aaltay @lukecwik
--
This is an automated message from the Apache Git Service
ibzib commented on pull request #16200:
URL: https://github.com/apache/beam/pull/16200#issuecomment-1007702822
Apologies - I've been a bit busy this week but I'll give this another review
next week.
--
This is an automated message from the Apache Git Service.
To respond to the message, p
aromanenko-dev merged pull request #16453:
URL: https://github.com/apache/beam/pull/16453
--
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-u
aromanenko-dev edited a comment on pull request #16454:
URL: https://github.com/apache/beam/pull/16454#issuecomment-1007591803
R: @mosche
--
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 spe
ihji merged pull request #16433:
URL: https://github.com/apache/beam/pull/16433
--
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-unsubscr...
tvalentyn merged pull request #16250:
URL: https://github.com/apache/beam/pull/16250
--
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-unsubs
codecov[bot] edited a comment on pull request #16456:
URL: https://github.com/apache/beam/pull/16456#issuecomment-1007619975
--
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.
lukecwik commented on pull request #16439:
URL: https://github.com/apache/beam/pull/16439#issuecomment-1007800054
Will get to this when I can, plat-mon has been busy.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
aromanenko-dev commented on pull request #16453:
URL: https://github.com/apache/beam/pull/16453#issuecomment-1007561419
R: @chamikaramj ptal
--
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 s
robertwb opened a new pull request #16463:
URL: https://github.com/apache/beam/pull/16463
I just ran `npx gts init` saying no to all options and then ran it on all
our files. We should probably do this when people don't have outstanding work.
Thank yo
robertwb closed pull request #16463:
URL: https://github.com/apache/beam/pull/16463
--
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-unsubsc
robertwb commented on pull request #16463:
URL: https://github.com/apache/beam/pull/16463#issuecomment-1008082328
Wrong repo.
--
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
robertwb opened a new pull request #16465:
URL: https://github.com/apache/beam/pull/16465
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly and easily:
- [ ] [**Choose
reviewer(s)**](https://b
robertwb commented on pull request #16465:
URL: https://github.com/apache/beam/pull/16465#issuecomment-1008230299
R: @katsiapis
--
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 comme
hmc-cs-mdrissi opened a new pull request #16466:
URL: https://github.com/apache/beam/pull/16466
Follow up to this [pr](https://github.com/apache/beam/pull/15233). A couple
months ago we loosen typing extensions to any typing extensions <4. Well,
typing extensions release a version 4 that i
codecov[bot] commented on pull request #16466:
URL: https://github.com/apache/beam/pull/16466#issuecomment-1008241452
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16466?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+
codecov[bot] edited a comment on pull request #16466:
URL: https://github.com/apache/beam/pull/16466#issuecomment-1008241452
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16466?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+
codecov[bot] edited a comment on pull request #16466:
URL: https://github.com/apache/beam/pull/16466#issuecomment-1008241452
#
[Codecov](https://codecov.io/gh/apache/beam/pull/16466?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+
60 matches
Mail list logo