kileys opened a new pull request #12899:
URL: https://github.com/apache/beam/pull/12899
**Please** add a meaningful description for your change here
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution
kileys commented on pull request #12899:
URL: https://github.com/apache/beam/pull/12899#issuecomment-696393175
Run Java PostCommit
This is an automated message from the Apache Git Service.
To respond to the message, please
lukecwik commented on pull request #11856:
URL: https://github.com/apache/beam/pull/11856#issuecomment-696392541
> Thanks @boyuanzz, it makes sense now why it didn't work. Unfortunately, I
can't use V2 runner, because I want to run these tests in batch mode as well
(V2 runner supports
TheNeuralBit commented on pull request #12896:
URL: https://github.com/apache/beam/pull/12896#issuecomment-696399245
Run Python_PVR_Flink PreCommit
This is an automated message from the Apache Git Service.
To respond to the
robertwb opened a new pull request #12900:
URL: https://github.com/apache/beam/pull/12900
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly and easily:
- [ ] [**Choose
codecov[bot] edited a comment on pull request #12896:
URL: https://github.com/apache/beam/pull/12896#issuecomment-696398122
# [Codecov](https://codecov.io/gh/apache/beam/pull/12896?src=pr=h1) Report
> Merging
[#12896](https://codecov.io/gh/apache/beam/pull/12896?src=pr=desc) into
boyuanzz commented on pull request #12888:
URL: https://github.com/apache/beam/pull/12888#issuecomment-696410670
Discussed with Luke and Cham separately, we can get ride of
`with_attributes` and `serialized_attribute_fn ` from both Read and Write.
There are corresponding changes for
codecov[bot] edited a comment on pull request #12880:
URL: https://github.com/apache/beam/pull/12880#issuecomment-695346831
# [Codecov](https://codecov.io/gh/apache/beam/pull/12880?src=pr=h1) Report
> Merging
[#12880](https://codecov.io/gh/apache/beam/pull/12880?src=pr=desc) into
pabloem commented on pull request #12762:
URL: https://github.com/apache/beam/pull/12762#issuecomment-696413272
Run Python 3.8 PostCommit
This is an automated message from the Apache Git Service.
To respond to the message,
trucleduc opened a new pull request #12902:
URL: https://github.com/apache/beam/pull/12902
**Please** add a meaningful description for your change here
In #12721 , we delete all FHIR stores in the clean up which may interfere
with other integration tests. This PR fixes this by only
TheNeuralBit commented on pull request #12902:
URL: https://github.com/apache/beam/pull/12902#issuecomment-696415795
Run Java PostCommit
This is an automated message from the Apache Git Service.
To respond to the message,
ibzib commented on a change in pull request #12637:
URL: https://github.com/apache/beam/pull/12637#discussion_r492386778
##
File path: sdks/python/apache_beam/runners/worker/data_plane_test.py
##
@@ -108,16 +106,28 @@ def send(instruction_id, transform_id, data):
])
trucleduc commented on pull request #12902:
URL: https://github.com/apache/beam/pull/12902#issuecomment-696417081
@TheNeuralBit Would you please hold on merging, I realized that there could
be flakiness issue for the existing tests. I'll investigate.
trucleduc commented on pull request #12902:
URL: https://github.com/apache/beam/pull/12902#issuecomment-696419895
@TheNeuralBit ok fixed. I tried to run the test 10 times and they all passed.
This is an automated message
TheNeuralBit commented on pull request #12896:
URL: https://github.com/apache/beam/pull/12896#issuecomment-696422086
Run Python_PVR_Flink PreCommit
This is an automated message from the Apache Git Service.
To respond to the
TheNeuralBit commented on pull request #12902:
URL: https://github.com/apache/beam/pull/12902#issuecomment-696421674
Run Java PostCommit
This is an automated message from the Apache Git Service.
To respond to the message,
aaltay commented on pull request #12892:
URL: https://github.com/apache/beam/pull/12892#issuecomment-696425242
/cc @KevinGG
This is an automated message from the Apache Git Service.
To respond to the message, please log on
codecov[bot] edited a comment on pull request #12727:
URL: https://github.com/apache/beam/pull/12727#issuecomment-683229966
# [Codecov](https://codecov.io/gh/apache/beam/pull/12727?src=pr=h1) Report
> Merging
[#12727](https://codecov.io/gh/apache/beam/pull/12727?src=pr=desc) into
codecov[bot] edited a comment on pull request #12799:
URL: https://github.com/apache/beam/pull/12799#issuecomment-692960218
# [Codecov](https://codecov.io/gh/apache/beam/pull/12799?src=pr=h1) Report
> Merging
[#12799](https://codecov.io/gh/apache/beam/pull/12799?src=pr=desc) into
pabloem commented on pull request #12782:
URL: https://github.com/apache/beam/pull/12782#issuecomment-696442124
Run Python 3.8 PostCommit
This is an automated message from the Apache Git Service.
To respond to the message,
codecov[bot] edited a comment on pull request #12727:
URL: https://github.com/apache/beam/pull/12727#issuecomment-683229966
# [Codecov](https://codecov.io/gh/apache/beam/pull/12727?src=pr=h1) Report
> Merging
[#12727](https://codecov.io/gh/apache/beam/pull/12727?src=pr=desc) into
lukecwik commented on pull request #12794:
URL: https://github.com/apache/beam/pull/12794#issuecomment-696393557
> Thanks for the feedback and the merge Luke. I'll address the updates in a
PR.
>
> I agree all tests makes more sense as one can never have enough coverage.
That said,
chamikaramj commented on a change in pull request #12611:
URL: https://github.com/apache/beam/pull/12611#discussion_r492364337
##
File path: sdks/python/apache_beam/io/gcp/spanner.py
##
@@ -0,0 +1,504 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
codecov[bot] edited a comment on pull request #12896:
URL: https://github.com/apache/beam/pull/12896#issuecomment-696398122
# [Codecov](https://codecov.io/gh/apache/beam/pull/12896?src=pr=h1) Report
> Merging
[#12896](https://codecov.io/gh/apache/beam/pull/12896?src=pr=desc) into
codecov[bot] commented on pull request #12896:
URL: https://github.com/apache/beam/pull/12896#issuecomment-696398122
# [Codecov](https://codecov.io/gh/apache/beam/pull/12896?src=pr=h1) Report
> Merging
[#12896](https://codecov.io/gh/apache/beam/pull/12896?src=pr=desc) into
robertwb commented on pull request #12873:
URL: https://github.com/apache/beam/pull/12873#issuecomment-696399851
Sure, we can keep it experimental a bit longer. I would like to fix
https://github.com/apache/beam/pull/12900 though.
ibzib opened a new pull request #12901:
URL: https://github.com/apache/beam/pull/12901
R: @amaliujia
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly and easily:
- [ ] [**Choose
TheNeuralBit commented on pull request #12721:
URL: https://github.com/apache/beam/pull/12721#issuecomment-696407136
I think this broke Fhir integration tests in PostCommit:
https://ci-beam.apache.org/job/beam_PostCommit_Java/6624/
trucleduc commented on pull request #12721:
URL: https://github.com/apache/beam/pull/12721#issuecomment-696408469
I guess the issue is because we delete all FHIR stores after test. I'll send
a PR to fix it.
kennknowles commented on pull request #12877:
URL: https://github.com/apache/beam/pull/12877#issuecomment-696412239
Ran `./gradlew -Ppublishing
-PjavaLinkageArtifactIds=beam-sdks-java-io-google-cloud-platform,beam-runners-google-cloud-dataflow-java
:checkJavaLinkage > ~/tmp/linkage-after`
tvalentyn commented on pull request #12895:
URL: https://github.com/apache/beam/pull/12895#issuecomment-696411840
Run Seed Job
This is an automated message from the Apache Git Service.
To respond to the message, please log
chamikaramj commented on a change in pull request #12880:
URL: https://github.com/apache/beam/pull/12880#discussion_r492380763
##
File path: sdks/python/apache_beam/runners/dataflow/dataflow_runner.py
##
@@ -488,12 +497,15 @@ def run_pipeline(self, pipeline, options):
#
trucleduc commented on pull request #12721:
URL: https://github.com/apache/beam/pull/12721#issuecomment-696414909
Sent https://github.com/apache/beam/pull/12902
This is an automated message from the Apache Git Service.
To
TheNeuralBit commented on pull request #12902:
URL: https://github.com/apache/beam/pull/12902#issuecomment-696417351
ok I'll hold off, thanks
This is an automated message from the Apache Git Service.
To respond to the
rohdesamuel commented on pull request #12799:
URL: https://github.com/apache/beam/pull/12799#issuecomment-696426921
Run Python PreCommit
This is an automated message from the Apache Git Service.
To respond to the message,
TheNeuralBit commented on pull request #12896:
URL: https://github.com/apache/beam/pull/12896#issuecomment-696430017
PVR_Flink is now disabled:
https://ci-beam.apache.org/job/beam_PreCommit_Python_PVR_Flink_Commit I don't
think the failure is related.
TheNeuralBit merged pull request #12896:
URL: https://github.com/apache/beam/pull/12896
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
ibzib commented on pull request #12896:
URL: https://github.com/apache/beam/pull/12896#issuecomment-696436679
> PVR_Flink is now disabled:
https://ci-beam.apache.org/job/beam_PreCommit_Python_PVR_Flink_Commit I don't
think the failure is related.
That's unexpected.. I wonder if it
codecov[bot] edited a comment on pull request #12727:
URL: https://github.com/apache/beam/pull/12727#issuecomment-683229966
# [Codecov](https://codecov.io/gh/apache/beam/pull/12727?src=pr=h1) Report
> Merging
[#12727](https://codecov.io/gh/apache/beam/pull/12727?src=pr=desc) into
codecov[bot] edited a comment on pull request #12889:
URL: https://github.com/apache/beam/pull/12889#issuecomment-696446965
# [Codecov](https://codecov.io/gh/apache/beam/pull/12889?src=pr=h1) Report
> Merging
[#12889](https://codecov.io/gh/apache/beam/pull/12889?src=pr=desc) into
codecov[bot] edited a comment on pull request #12900:
URL: https://github.com/apache/beam/pull/12900#issuecomment-696448479
# [Codecov](https://codecov.io/gh/apache/beam/pull/12900?src=pr=h1) Report
> Merging
[#12900](https://codecov.io/gh/apache/beam/pull/12900?src=pr=desc) into
chamikaramj commented on pull request #12880:
URL: https://github.com/apache/beam/pull/12880#issuecomment-696451661
Run Python_PVR_Flink PreCommit
This is an automated message from the Apache Git Service.
To respond to the
pabloem commented on pull request #12762:
URL: https://github.com/apache/beam/pull/12762#issuecomment-696456903
Run Python 3.8 PostCommit
This is an automated message from the Apache Git Service.
To respond to the message,
pabloem commented on pull request #12782:
URL: https://github.com/apache/beam/pull/12782#issuecomment-696456995
Run Python2_PVR_Flink PreCommit
This is an automated message from the Apache Git Service.
To respond to the
codecov[bot] edited a comment on pull request #12880:
URL: https://github.com/apache/beam/pull/12880#issuecomment-695346831
# [Codecov](https://codecov.io/gh/apache/beam/pull/12880?src=pr=h1) Report
> Merging
[#12880](https://codecov.io/gh/apache/beam/pull/12880?src=pr=desc) into
codecov[bot] edited a comment on pull request #12880:
URL: https://github.com/apache/beam/pull/12880#issuecomment-695346831
# [Codecov](https://codecov.io/gh/apache/beam/pull/12880?src=pr=h1) Report
> Merging
[#12880](https://codecov.io/gh/apache/beam/pull/12880?src=pr=desc) into
chamikaramj merged pull request #12880:
URL: https://github.com/apache/beam/pull/12880
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
codecov[bot] edited a comment on pull request #12880:
URL: https://github.com/apache/beam/pull/12880#issuecomment-695346831
# [Codecov](https://codecov.io/gh/apache/beam/pull/12880?src=pr=h1) Report
> Merging
[#12880](https://codecov.io/gh/apache/beam/pull/12880?src=pr=desc) into
codecov[bot] edited a comment on pull request #9907:
URL: https://github.com/apache/beam/pull/9907#issuecomment-682206351
# [Codecov](https://codecov.io/gh/apache/beam/pull/9907?src=pr=h1) Report
> Merging
[#9907](https://codecov.io/gh/apache/beam/pull/9907?src=pr=desc) into
tvalentyn commented on pull request #12895:
URL: https://github.com/apache/beam/pull/12895#issuecomment-696467526
Run Python Flink ValidatesRunner
This is an automated message from the Apache Git Service.
To respond to the
aaltay commented on a change in pull request #9907:
URL: https://github.com/apache/beam/pull/9907#discussion_r492447574
##
File path: sdks/python/apache_beam/options/pipeline_options.py
##
@@ -476,6 +499,26 @@ def _add_argparse_args(cls, parser):
'time. NOTE: only
kennknowles commented on pull request #12877:
URL: https://github.com/apache/beam/pull/12877#issuecomment-696412239
Ran `./gradlew -Ppublishing
-PjavaLinkageArtifactIds=beam-sdks-java-io-google-cloud-platform,beam-runners-google-cloud-dataflow-java
:checkJavaLinkage > ~/tmp/linkage-after`
robertwb commented on pull request #12873:
URL: https://github.com/apache/beam/pull/12873#issuecomment-696399851
Sure, we can keep it experimental a bit longer. I would like to fix
https://github.com/apache/beam/pull/12900 though.
codecov[bot] edited a comment on pull request #12889:
URL: https://github.com/apache/beam/pull/12889#issuecomment-696446965
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
kileys commented on pull request #12899:
URL: https://github.com/apache/beam/pull/12899#issuecomment-696393175
Run Java PostCommit
This is an automated message from the Apache Git Service.
To respond to the message, please
TheNeuralBit merged pull request #12896:
URL: https://github.com/apache/beam/pull/12896
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
kennknowles merged pull request #12875:
URL: https://github.com/apache/beam/pull/12875
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
udim commented on pull request #12893:
URL: https://github.com/apache/beam/pull/12893#issuecomment-696321660
R: @chamikaramj
This is an automated message from the Apache Git Service.
To respond to the message, please log on
y1chi commented on a change in pull request #12727:
URL: https://github.com/apache/beam/pull/12727#discussion_r492220801
##
File path: sdks/python/apache_beam/transforms/environments.py
##
@@ -252,6 +254,14 @@ def from_runner_api_parameter(payload, capabilities,
artifacts,
ihji commented on pull request #12822:
URL: https://github.com/apache/beam/pull/12822#issuecomment-696326864
@ajamato PTAL.
This is an automated message from the Apache Git Service.
To respond to the message, please log on
robertwb merged pull request #12889:
URL: https://github.com/apache/beam/pull/12889
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
ibzib merged pull request #12876:
URL: https://github.com/apache/beam/pull/12876
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
chamikaramj commented on pull request #12880:
URL: https://github.com/apache/beam/pull/12880#issuecomment-696230350
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
TheNeuralBit commented on a change in pull request #12830:
URL: https://github.com/apache/beam/pull/12830#discussion_r492339081
##
File path:
sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/pubsub/TestPubsubSignal.java
##
@@ -239,6 +245,12 @@
TheNeuralBit commented on pull request #12721:
URL: https://github.com/apache/beam/pull/12721#issuecomment-696407136
I think this broke Fhir integration tests in PostCommit:
https://ci-beam.apache.org/job/beam_PostCommit_Java/6624/
TheNeuralBit commented on a change in pull request #12882:
URL: https://github.com/apache/beam/pull/12882#discussion_r492219317
##
File path: sdks/python/apache_beam/dataframe/schemas.py
##
@@ -55,17 +159,149 @@ def expand(self, pcoll):
lambda batch:
aromanenko-dev commented on pull request #11459:
URL: https://github.com/apache/beam/pull/11459#issuecomment-696273839
The only minor thing that is missing - update of `CHANGES.md` about this
new IO.
This is an automated
lukecwik commented on pull request #12367:
URL: https://github.com/apache/beam/pull/12367#issuecomment-696223362
We have a merge on green policy so I have been rerunning the tests to get
past a known issue.
This is an
chamikaramj commented on a change in pull request #12611:
URL: https://github.com/apache/beam/pull/12611#discussion_r492364337
##
File path: sdks/python/apache_beam/io/gcp/spanner.py
##
@@ -0,0 +1,504 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
abhiy13 commented on a change in pull request #12645:
URL: https://github.com/apache/beam/pull/12645#discussion_r492474752
##
File path:
sdks/java/io/contextual-text-io/src/main/java/org/apache/beam/sdk/io/contextualtextio/RecordWithMetadata.java
##
@@ -0,0 +1,85 @@
+/*
+ *
abhiy13 commented on a change in pull request #12645:
URL: https://github.com/apache/beam/pull/12645#discussion_r492474814
##
File path:
sdks/java/io/contextual-text-io/src/main/java/org/apache/beam/sdk/io/contextualtextio/ContextualTextIO.java
##
@@ -0,0 +1,631 @@
+/*
+ *
abhiy13 commented on a change in pull request #12645:
URL: https://github.com/apache/beam/pull/12645#discussion_r492475231
##
File path: sdks/java/io/contextual-text-io/build.gradle
##
@@ -0,0 +1,41 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or
abhiy13 commented on a change in pull request #12645:
URL: https://github.com/apache/beam/pull/12645#discussion_r492474704
##
File path:
sdks/java/io/contextual-text-io/src/main/java/org/apache/beam/sdk/io/contextualtextio/ContextualTextIOSource.java
##
@@ -0,0 +1,364 @@
+/*
abhiy13 commented on a change in pull request #12645:
URL: https://github.com/apache/beam/pull/12645#discussion_r492474778
##
File path:
sdks/java/io/contextual-text-io/src/main/java/org/apache/beam/sdk/io/contextualtextio/ContextualTextIO.java
##
@@ -0,0 +1,631 @@
+/*
+ *
robertwb commented on a change in pull request #12884:
URL: https://github.com/apache/beam/pull/12884#discussion_r492465887
##
File path: sdks/python/apache_beam/coders/coder_impl.py
##
@@ -725,7 +728,7 @@ def __init__(self, key_coder_impl, window_coder_impl):
codecov[bot] edited a comment on pull request #12645:
URL: https://github.com/apache/beam/pull/12645#issuecomment-688630083
# [Codecov](https://codecov.io/gh/apache/beam/pull/12645?src=pr=h1) Report
> :exclamation: No coverage uploaded for pull request base
(`master@2b2b8e7`). [Click
robertwb commented on pull request #12884:
URL: https://github.com/apache/beam/pull/12884#issuecomment-696504555
Run Python PreCommit
This is an automated message from the Apache Git Service.
To respond to the message,
robertwb commented on a change in pull request #12881:
URL: https://github.com/apache/beam/pull/12881#discussion_r492466948
##
File path: sdks/python/apache_beam/runners/worker/bundle_processor.py
##
@@ -119,10 +125,10 @@ class RunnerIOOperation(operations.Operation):
def
codecov[bot] commented on pull request #12884:
URL: https://github.com/apache/beam/pull/12884#issuecomment-696510544
# [Codecov](https://codecov.io/gh/apache/beam/pull/12884?src=pr=h1) Report
> Merging
[#12884](https://codecov.io/gh/apache/beam/pull/12884?src=pr=desc) into
codecov[bot] edited a comment on pull request #12889:
URL: https://github.com/apache/beam/pull/12889#issuecomment-696446965
# [Codecov](https://codecov.io/gh/apache/beam/pull/12889?src=pr=h1) Report
> Merging
[#12889](https://codecov.io/gh/apache/beam/pull/12889?src=pr=desc) into
codecov[bot] commented on pull request #12762:
URL: https://github.com/apache/beam/pull/12762#issuecomment-696459985
# [Codecov](https://codecov.io/gh/apache/beam/pull/12762?src=pr=h1) Report
> Merging
[#12762](https://codecov.io/gh/apache/beam/pull/12762?src=pr=desc) into
codecov[bot] commented on pull request #12889:
URL: https://github.com/apache/beam/pull/12889#issuecomment-696446965
# [Codecov](https://codecov.io/gh/apache/beam/pull/12889?src=pr=h1) Report
> Merging
[#12889](https://codecov.io/gh/apache/beam/pull/12889?src=pr=desc) into
chamikaramj commented on pull request #12880:
URL: https://github.com/apache/beam/pull/12880#issuecomment-696447942
Run Python_PVR_Flink PreCommit
This is an automated message from the Apache Git Service.
To respond to the
robertwb merged pull request #12889:
URL: https://github.com/apache/beam/pull/12889
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
udim commented on pull request #9907:
URL: https://github.com/apache/beam/pull/9907#issuecomment-696465531
PTAL, changes are in the last commit.
This is an automated message from the Apache Git Service.
To respond to the
codecov[bot] edited a comment on pull request #12645:
URL: https://github.com/apache/beam/pull/12645#issuecomment-688630083
# [Codecov](https://codecov.io/gh/apache/beam/pull/12645?src=pr=h1) Report
> :exclamation: No coverage uploaded for pull request base
(`master@2b2b8e7`). [Click
abhiy13 commented on a change in pull request #12645:
URL: https://github.com/apache/beam/pull/12645#discussion_r492475183
##
File path:
sdks/java/io/contextual-text-io/src/main/java/org/apache/beam/sdk/io/contextualtextio/RecordWithMetadata.java
##
@@ -0,0 +1,84 @@
+/*
+ *
robertwb commented on a change in pull request #12727:
URL: https://github.com/apache/beam/pull/12727#discussion_r492422441
##
File path: sdks/python/apache_beam/runners/portability/stager.py
##
@@ -119,6 +119,7 @@ def create_job_resources(options, # type: PipelineOptions
tvalentyn commented on pull request #12895:
URL: https://github.com/apache/beam/pull/12895#issuecomment-696458288
Run Seed Job
This is an automated message from the Apache Git Service.
To respond to the message, please log
codecov[bot] edited a comment on pull request #12880:
URL: https://github.com/apache/beam/pull/12880#issuecomment-695346831
# [Codecov](https://codecov.io/gh/apache/beam/pull/12880?src=pr=h1) Report
> Merging
[#12880](https://codecov.io/gh/apache/beam/pull/12880?src=pr=desc) into
codecov[bot] edited a comment on pull request #12900:
URL: https://github.com/apache/beam/pull/12900#issuecomment-696448479
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
tvalentyn commented on a change in pull request #12895:
URL: https://github.com/apache/beam/pull/12895#discussion_r492360722
##
File path: sdks/python/test-suites/gradle.properties
##
@@ -27,3 +27,6 @@ dataflow_chicago_taxi_example_task_py_versions=3.7
# direct test-suites
ibzib commented on a change in pull request #12637:
URL: https://github.com/apache/beam/pull/12637#discussion_r492386778
##
File path: sdks/python/apache_beam/runners/worker/data_plane_test.py
##
@@ -108,16 +106,28 @@ def send(instruction_id, transform_id, data):
])
TheNeuralBit commented on pull request #12896:
URL: https://github.com/apache/beam/pull/12896#issuecomment-696446062
I think that job is actually a new one added in
https://github.com/apache/beam/pull/12898. There's still
codecov[bot] edited a comment on pull request #12900:
URL: https://github.com/apache/beam/pull/12900#issuecomment-696448479
# [Codecov](https://codecov.io/gh/apache/beam/pull/12900?src=pr=h1) Report
> Merging
[#12900](https://codecov.io/gh/apache/beam/pull/12900?src=pr=desc) into
tvalentyn commented on pull request #12896:
URL: https://github.com/apache/beam/pull/12896#issuecomment-696457581
> That's unexpected.. I wonder if it was disabled accidentally (perhaps
because of Python version removals?)
Yes, sorry about that - I was testing my changes and ran a
codecov[bot] edited a comment on pull request #12880:
URL: https://github.com/apache/beam/pull/12880#issuecomment-695346831
# [Codecov](https://codecov.io/gh/apache/beam/pull/12880?src=pr=h1) Report
> Merging
[#12880](https://codecov.io/gh/apache/beam/pull/12880?src=pr=desc) into
codecov[bot] edited a comment on pull request #9907:
URL: https://github.com/apache/beam/pull/9907#issuecomment-682206351
# [Codecov](https://codecov.io/gh/apache/beam/pull/9907?src=pr=h1) Report
> Merging
[#9907](https://codecov.io/gh/apache/beam/pull/9907?src=pr=desc) into
lukecwik commented on a change in pull request #12637:
URL: https://github.com/apache/beam/pull/12637#discussion_r492359619
##
File path: sdks/python/apache_beam/runners/worker/data_plane_test.py
##
@@ -108,16 +106,28 @@ def send(instruction_id, transform_id, data):
tvalentyn commented on pull request #12898:
URL: https://github.com/apache/beam/pull/12898#issuecomment-696385996
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
1 - 100 of 341 matches
Mail list logo