reuvenlax merged pull request #11725:
URL: https://github.com/apache/beam/pull/11725
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
tvalentyn commented on pull request #11726:
URL: https://github.com/apache/beam/pull/11726#issuecomment-629590663
R: @chamikaramj
This is an automated message from the Apache Git Service.
To respond to the message, please lo
youngoli merged pull request #11728:
URL: https://github.com/apache/beam/pull/11728
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
youngoli commented on a change in pull request #11728:
URL: https://github.com/apache/beam/pull/11728#discussion_r426104694
##
File path: sdks/go/pkg/beam/io/synthetic/step.go
##
@@ -0,0 +1,191 @@
+// Licensed to the Apache Software Foundation (ASF) under one or more
+// contri
youngoli commented on a change in pull request #11728:
URL: https://github.com/apache/beam/pull/11728#discussion_r426103916
##
File path: sdks/go/pkg/beam/io/synthetic/step.go
##
@@ -0,0 +1,191 @@
+// Licensed to the Apache Software Foundation (ASF) under one or more
+// contri
ibzib opened a new pull request #11729:
URL: https://github.com/apache/beam/pull/11729
This should not be merged until the 2.21.0 release is finalized.
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quic
ibzib merged pull request #11717:
URL: https://github.com/apache/beam/pull/11717
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
aaltay commented on pull request #11437:
URL: https://github.com/apache/beam/pull/11437#issuecomment-629549790
Run Java_Examples_Dataflow PreCommit
This is an automated message from the Apache Git Service.
To respond to the m
aaltay commented on pull request #11437:
URL: https://github.com/apache/beam/pull/11437#issuecomment-629549669
Run Java_Examples_Dataflow PreCommit
This is an automated message from the Apache Git Service.
To respond to the m
rehmanmuradali commented on pull request #11725:
URL: https://github.com/apache/beam/pull/11725#issuecomment-629545286
LGTM. Thanks for taking care of it.
This is an automated message from the Apache Git Service.
To respond t
lostluck commented on a change in pull request #11728:
URL: https://github.com/apache/beam/pull/11728#discussion_r426072025
##
File path: sdks/go/pkg/beam/io/synthetic/step.go
##
@@ -0,0 +1,191 @@
+// Licensed to the Apache Software Foundation (ASF) under one or more
+// contri
reuvenlax commented on pull request #11725:
URL: https://github.com/apache/beam/pull/11725#issuecomment-629539505
run dataflow validatesrunner
This is an automated message from the Apache Git Service.
To respond to the messag
reuvenlax commented on pull request #11725:
URL: https://github.com/apache/beam/pull/11725#issuecomment-629539400
R: @rehmanmuradali
This is an automated message from the Apache Git Service.
To respond to the message, please
ibzib commented on a change in pull request #11708:
URL: https://github.com/apache/beam/pull/11708#discussion_r426075525
##
File path: sdks/python/apache_beam/runners/portability/abstract_job_service.py
##
@@ -288,6 +289,9 @@ def close(self):
self._zipfile_handle.close()
kennknowles commented on a change in pull request #10946:
URL: https://github.com/apache/beam/pull/10946#discussion_r426073628
##
File path:
sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/rel/BeamTableFunctionScanRel.java
##
@@ -0,0 +1,162 @@
+/
youngoli commented on pull request #11728:
URL: https://github.com/apache/beam/pull/11728#issuecomment-629520803
R: @lostluck
This is an automated message from the Apache Git Service.
To respond to the message, please log on
chamikaramj edited a comment on pull request #11039:
URL: https://github.com/apache/beam/pull/11039#issuecomment-629519752
Yes, I think we need this. Heejong, can you make the JIRA a blocker for beam
2.22 ?
This is an automa
chamikaramj commented on pull request #11039:
URL: https://github.com/apache/beam/pull/11039#issuecomment-629519752
Yes, I think we need this. Heejong, can you make the jar a blocker for beam
2.22 ?
This is an automated mess
youngoli opened a new pull request #11728:
URL: https://github.com/apache/beam/pull/11728
Building off the synthetic source, adding a synthetic step. Still need
to update the way the configs are built though.
Thank you for your contribution! Follow t
robertwb commented on a change in pull request #11708:
URL: https://github.com/apache/beam/pull/11708#discussion_r426061393
##
File path: sdks/python/apache_beam/runners/portability/abstract_job_service.py
##
@@ -313,39 +329,62 @@ def prepare(self):
with tempfile.NamedTemp
aaltay commented on pull request #11437:
URL: https://github.com/apache/beam/pull/11437#issuecomment-629507774
retest this please
This is an automated message from the Apache Git Service.
To respond to the message, please log
ibzib opened a new pull request #11727:
URL: https://github.com/apache/beam/pull/11727
This PR should only be merged after the 2.21.0 release is finalized.
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution
youngoli merged pull request #11665:
URL: https://github.com/apache/beam/pull/11665
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
aaltay commented on pull request #11437:
URL: https://github.com/apache/beam/pull/11437#issuecomment-629504732
retest this please
This is an automated message from the Apache Git Service.
To respond to the message, please log
chadrik commented on pull request #11070:
URL: https://github.com/apache/beam/pull/11070#issuecomment-629498648
> Meanwhile, if you make suggestions I think that'd be easiest. (I hope)
Done!
This is an automated me
chadrik commented on a change in pull request #11070:
URL: https://github.com/apache/beam/pull/11070#discussion_r426055296
##
File path: website/src/_posts/2020-03-06-python-typing.md
##
@@ -0,0 +1,117 @@
+---
+layout: post
+title: "Python SDK Typing Changes"
+date: 2020-03-
chadrik commented on a change in pull request #11070:
URL: https://github.com/apache/beam/pull/11070#discussion_r426054969
##
File path: website/src/_posts/2020-03-06-python-typing.md
##
@@ -0,0 +1,117 @@
+---
+layout: post
+title: "Python SDK Typing Changes"
+date: 2020-03-
chadrik commented on a change in pull request #11070:
URL: https://github.com/apache/beam/pull/11070#discussion_r426055142
##
File path: website/src/_posts/2020-03-06-python-typing.md
##
@@ -0,0 +1,117 @@
+---
+layout: post
+title: "Python SDK Typing Changes"
+date: 2020-03-
rezarokni commented on pull request #11437:
URL: https://github.com/apache/beam/pull/11437#issuecomment-629487986
added back the stuff removed by merge , so hopefully this can now go in.
And then I will raise PR num two for the website changes.
--
tvalentyn removed a comment on pull request #11726:
URL: https://github.com/apache/beam/pull/11726#issuecomment-629478846
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
tvalentyn removed a comment on pull request #11726:
URL: https://github.com/apache/beam/pull/11726#issuecomment-629478755
Run Python2 PostCommit
This is an automated message from the Apache Git Service.
To respond to the mess
rezarokni commented on pull request #11437:
URL: https://github.com/apache/beam/pull/11437#issuecomment-629484336
Merge got rid of changes to snippits.java adding that back
This is an automated message from the Apache Git Ser
tvalentyn commented on pull request #11726:
URL: https://github.com/apache/beam/pull/11726#issuecomment-629478942
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
tvalentyn commented on pull request #11726:
URL: https://github.com/apache/beam/pull/11726#issuecomment-629478755
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
tvalentyn opened a new pull request #11726:
URL: https://github.com/apache/beam/pull/11726
Post-Commit Tests Status (on master branch)
Lang | SDK | Apex | Dataflow | Flink | Gearpu
pabloem merged pull request #11634:
URL: https://github.com/apache/beam/pull/11634
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 t
pabloem commented on pull request #11634:
URL: https://github.com/apache/beam/pull/11634#issuecomment-629477820
LGTM
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
pabloem merged pull request #11710:
URL: https://github.com/apache/beam/pull/11710
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 t
pabloem commented on pull request #11710:
URL: https://github.com/apache/beam/pull/11710#issuecomment-629476263
thanks @omarismail94 !
This is an automated message from the Apache Git Service.
To respond to the message, pleas
amaliujia commented on pull request #11610:
URL: https://github.com/apache/beam/pull/11610#issuecomment-629468556
retest this please
This is an automated message from the Apache Git Service.
To respond to the message, please
pabloem merged pull request #11719:
URL: https://github.com/apache/beam/pull/11719
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 t
lukecwik merged pull request #11721:
URL: https://github.com/apache/beam/pull/11721
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
pabloem commented on pull request #11719:
URL: https://github.com/apache/beam/pull/11719#issuecomment-629456417
thank you @bntnam !
This is an automated message from the Apache Git Service.
To respond to the message, please l
tvalentyn commented on pull request #11707:
URL: https://github.com/apache/beam/pull/11707#issuecomment-629456098
re:
> I didn't change py37-lint to py38-lint, because this requires updating
dependencies (for example, our current version of pycodestyle doesn't support
Python 3.8
ananvay commented on pull request #11720:
URL: https://github.com/apache/beam/pull/11720#issuecomment-629455295
LGTM. Thanks Robert!
This is an automated message from the Apache Git Service.
To respond to the message, please
robertwb commented on a change in pull request #11720:
URL: https://github.com/apache/beam/pull/11720#discussion_r426020019
##
File path:
sdks/java/harness/src/main/java/org/apache/beam/fn/harness/FnHarness.java
##
@@ -206,18 +205,16 @@ public static void main(
Loadin
pabloem commented on pull request #11710:
URL: https://github.com/apache/beam/pull/11710#issuecomment-629454663
Run Java PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, please lo
robertwb commented on a change in pull request #11720:
URL: https://github.com/apache/beam/pull/11720#discussion_r426018974
##
File path:
sdks/java/harness/src/main/java/org/apache/beam/fn/harness/FnHarness.java
##
@@ -206,18 +205,16 @@ public static void main(
Loadin
pabloem commented on pull request #11719:
URL: https://github.com/apache/beam/pull/11719#issuecomment-629452893
retest this please
This is an automated message from the Apache Git Service.
To respond to the message, please lo
ananvay commented on pull request #11720:
URL: https://github.com/apache/beam/pull/11720#issuecomment-629447440
Thanks a lot Robert! Overall LGTM. Just a few minor comments, PTAL.
This is an automated message from the Apache
ananvay commented on a change in pull request #11720:
URL: https://github.com/apache/beam/pull/11720#discussion_r426013359
##
File path:
sdks/java/harness/src/main/java/org/apache/beam/fn/harness/FnHarness.java
##
@@ -206,18 +205,16 @@ public static void main(
Loading
ananvay commented on a change in pull request #11720:
URL: https://github.com/apache/beam/pull/11720#discussion_r426012824
##
File path:
sdks/java/harness/src/main/java/org/apache/beam/fn/harness/FnHarness.java
##
@@ -206,18 +205,16 @@ public static void main(
Loading
apilloud merged pull request #11698:
URL: https://github.com/apache/beam/pull/11698
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
apilloud commented on pull request #11698:
URL: https://github.com/apache/beam/pull/11698#issuecomment-629438381
At very least you left comments on the bug.
This is an automated message from the Apache Git Service.
To respond
pabloem commented on pull request #11086:
URL: https://github.com/apache/beam/pull/11086#issuecomment-629436995
Run Python 3.7 PostCommit
This is an automated message from the Apache Git Service.
To respond to the message, pl
boyuanzz commented on a change in pull request #11715:
URL: https://github.com/apache/beam/pull/11715#discussion_r425995961
##
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/transforms/splittabledofn/GrowableOffsetRangeTracker.java
##
@@ -0,0 +1,103 @@
+/*
+ * Lic
pabloem commented on pull request #11339:
URL: https://github.com/apache/beam/pull/11339#issuecomment-629426161
retest this please
This is an automated message from the Apache Git Service.
To respond to the message, please lo
pabloem commented on pull request #11712:
URL: https://github.com/apache/beam/pull/11712#issuecomment-629425913
lgtm given that the we can get precommit to pass before merging
This is an automated message from the Apache Git
TheNeuralBit commented on pull request #11272:
URL: https://github.com/apache/beam/pull/11272#issuecomment-629422818
Run Java PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, plea
reuvenlax commented on pull request #11725:
URL: https://github.com/apache/beam/pull/11725#issuecomment-629422623
run dataflow validatesrunner
This is an automated message from the Apache Git Service.
To respond to the messag
reuvenlax opened a new pull request #11725:
URL: https://github.com/apache/beam/pull/11725
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 a change in pull request #11708:
URL: https://github.com/apache/beam/pull/11708#discussion_r425987361
##
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/io/ClassLoaderFileSystem.java
##
@@ -0,0 +1,154 @@
+/*
+ * Licensed to the Apache Software Fo
ibzib commented on a change in pull request #11708:
URL: https://github.com/apache/beam/pull/11708#discussion_r425986977
##
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/io/ClassLoaderFileSystem.java
##
@@ -0,0 +1,161 @@
+/*
+ * Licensed to the Apache Software Fo
TheNeuralBit commented on pull request #11272:
URL: https://github.com/apache/beam/pull/11272#issuecomment-629421832
Run Java PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, plea
boyuanzz commented on a change in pull request #11715:
URL: https://github.com/apache/beam/pull/11715#discussion_r425986350
##
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/transforms/splittabledofn/GrowableOffsetRangeTracker.java
##
@@ -0,0 +1,103 @@
+/*
+ * Lic
robinyqiu commented on pull request #11272:
URL: https://github.com/apache/beam/pull/11272#issuecomment-629420575
Java PreCommit failed due to a build failure. Please help run again.
This is an automated message from the Apac
lukecwik commented on pull request #11721:
URL: https://github.com/apache/beam/pull/11721#issuecomment-629419737
Run Java PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, please l
aaltay opened a new pull request #11723:
URL: https://github.com/apache/beam/pull/11723
**Please** add a meaningful description for your change here
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly
aaltay closed pull request #11723:
URL: https://github.com/apache/beam/pull/11723
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 th
aaltay opened a new pull request #11724:
URL: https://github.com/apache/beam/pull/11724
**Please** add a meaningful description for your change here
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly
ananvay commented on pull request #11721:
URL: https://github.com/apache/beam/pull/11721#issuecomment-629411984
Thanks Luke! LGTM
This is an automated message from the Apache Git Service.
To respond to the message, please log
Hannah-Jiang commented on pull request #11717:
URL: https://github.com/apache/beam/pull/11717#issuecomment-629411176
> R: @ibzib
Oops, thank you.
This is an automated message from the Apache Git Service.
To respond to
Hannah-Jiang merged pull request #11584:
URL: https://github.com/apache/beam/pull/11584
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
jaketf commented on pull request #11596:
URL: https://github.com/apache/beam/pull/11596#issuecomment-629406986
@pabloem bump ^. please retest this.
This is an automated message from the Apache Git Service.
To respond to the m
ibzib commented on pull request #11722:
URL: https://github.com/apache/beam/pull/11722#issuecomment-629406842
> But I would not recommend redoing the RC if you are already done. Just
merge it to the branch and include it if you have to roll another RC.
Thanks Thomas. I have already f
tweise commented on pull request #11722:
URL: https://github.com/apache/beam/pull/11722#issuecomment-629405793
We are internally running 2.21 on Flink 1.10.1. There is little to no risk
including this.
But I would not recommend redoing the RC if you are already done. Just merge
it
amaliujia commented on pull request #10946:
URL: https://github.com/apache/beam/pull/10946#issuecomment-629403702
Run Java PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, please
ibzib commented on pull request #11722:
URL: https://github.com/apache/beam/pull/11722#issuecomment-629401614
I'm nearly finished making a release candidate. Is there an important reason
to cherry-pick this?
This is an autom
mxm commented on pull request #11678:
URL: https://github.com/apache/beam/pull/11678#issuecomment-629396868
Based on the load test results, I'm not changing the default behavior here.
Instead, flushing the bundle output before a checkpoint can be turned on
optionally.
---
lukecwik edited a comment on pull request #11721:
URL: https://github.com/apache/beam/pull/11721#issuecomment-629393804
R: @ananvay @youngoli
This is an automated message from the Apache Git Service.
To respond to the messag
aaltay commented on pull request #11437:
URL: https://github.com/apache/beam/pull/11437#issuecomment-629396682
retest this please
This is an automated message from the Apache Git Service.
To respond to the message, please log
tweise opened a new pull request #11722:
URL: https://github.com/apache/beam/pull/11722
Backport of #11683
Post-Commit Tests Status (on master branch)
Lang | SDK | Apex | Da
robertwb commented on pull request #11039:
URL: https://github.com/apache/beam/pull/11039#issuecomment-629394803
@ihji Just thinking about timing, were we hoping to get this into 2.22?
This is an automated message from the A
lukecwik commented on pull request #11721:
URL: https://github.com/apache/beam/pull/11721#issuecomment-629393804
R: @ananvay
This is an automated message from the Apache Git Service.
To respond to the message, please log on
aaltay commented on pull request #11437:
URL: https://github.com/apache/beam/pull/11437#issuecomment-629386383
retest this please
This is an automated message from the Apache Git Service.
To respond to the message, please log
lukecwik opened a new pull request #11721:
URL: https://github.com/apache/beam/pull/11721
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly and easily:
- [ ] [**Choose
reviewer(s)**](https://beam.
stale[bot] commented on pull request #10528:
URL: https://github.com/apache/beam/pull/10528#issuecomment-629380085
This pull request has been marked as stale due to 60 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
jaketf commented on pull request #11339:
URL: https://github.com/apache/beam/pull/11339#issuecomment-629376974
@pabloem 843104d was a pretty simple fix for api surface test
```
git rev-parse HEAD && ./gradlew
:sdks:java:io:google-cloud-platform:cleanTest
:sdks:java:io:google-cloud-pl
robertwb edited a comment on pull request #11720:
URL: https://github.com/apache/beam/pull/11720#issuecomment-629376271
R: @ananvay
This is an automated message from the Apache Git Service.
To respond to the message, please l
robertwb commented on pull request #11720:
URL: https://github.com/apache/beam/pull/11720#issuecomment-629376271
R: @ ananvay
This is an automated message from the Apache Git Service.
To respond to the message, please log on
robertwb opened a new pull request #11720:
URL: https://github.com/apache/beam/pull/11720
Instead always pull bundle descriptors from the runner.
This avoids having potentially duplicated caches of all bundle
Thank you for your contribution!
aaltay commented on pull request #11584:
URL: https://github.com/apache/beam/pull/11584#issuecomment-629369467
Run Python PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, please l
aaltay commented on pull request #11717:
URL: https://github.com/apache/beam/pull/11717#issuecomment-629369332
R: @ibzib
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
aaltay commented on pull request #11705:
URL: https://github.com/apache/beam/pull/11705#issuecomment-629368448
> stepik is the platform where these courses are uploaded , an downloaded by
intellij
Thank you! :)
This i
jaketf commented on pull request #11339:
URL: https://github.com/apache/beam/pull/11339#issuecomment-629367380
@pabloem re: api surface.
So the history here is that the client library executeBundle method is
broken so we were forced to use out own http client (and our own exceptions).
jaketf commented on pull request #11339:
URL: https://github.com/apache/beam/pull/11339#issuecomment-629364334
@mwalenia
> What exactly causes this
error?https://builds.apache.org/job/beam_PostCommit_Java_PR/362/testReport/junit/org.apache.beam.sdk.io.gcp.healthcare/FhirIOWriteIT/testFh
udim commented on pull request #11070:
URL: https://github.com/apache/beam/pull/11070#issuecomment-629360659
> > @chadrik, do you have the "edit file" option available?
>
> Nope :(
I think that maybe it's because you're not on
https://github.com/orgs/apache/teams/beam-committe
pabloem commented on pull request #11339:
URL: https://github.com/apache/beam/pull/11339#issuecomment-629357697
this test checks that we're not increasing the dependencies that we expose
in the API. We can add the Healthcare IO classes.
Other classes I think we should try to figure out h
pabloem commented on pull request #11339:
URL: https://github.com/apache/beam/pull/11339#issuecomment-629356272
We have a failure from a test checking out the API surface for the gcp io
package:
https://builds.apache.org/job/beam_PreCommit_Java_Commit/11386/testReport/junit/org.apache.beam
mxm commented on pull request #11678:
URL: https://github.com/apache/beam/pull/11678#issuecomment-629353811
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
1 - 100 of 140 matches
Mail list logo