TobKed commented on pull request #1:
URL: https://github.com/apache/beam/pull/1#issuecomment-658002752
I checked actions tab in repository which I recently forked and I got
information and question about enabling workflows.
"elem3",
"elem2"
KevinGG opened a new pull request #12249:
URL: https://github.com/apache/beam/pull/12249
1. Interactive Beam now creates 1 cache manager for each user defined
pipeline.
2. Cache managers are created lazily when they are used.
3. Cleanup of cache and states are carried out independentl
apilloud commented on pull request #12238:
URL: https://github.com/apache/beam/pull/12238#issuecomment-657896739
Run Javadoc PostCommit
This is an automated message from the Apache Git Service.
To respond to the message, plea
davidcavazos commented on pull request #12217:
URL: https://github.com/apache/beam/pull/12217#issuecomment-657896586
R: @aaltay
This is an automated message from the Apache Git Service.
To respond to the message, please log
rosetn commented on pull request #12217:
URL: https://github.com/apache/beam/pull/12217#issuecomment-657894258
LGTM. Thank you, David!
This is an automated message from the Apache Git Service.
To respond to the message, pleas
pabloem commented on a change in pull request #12074:
URL: https://github.com/apache/beam/pull/12074#discussion_r447317739
##
File path: sdks/python/apache_beam/transforms/combiners.py
##
@@ -62,26 +62,28 @@
TimestampType = Union[int, float, Timestamp, Duration]
+class Com
robertwb opened a new pull request #12248:
URL: https://github.com/apache/beam/pull/12248
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 #12248:
URL: https://github.com/apache/beam/pull/12248#issuecomment-657884514
R: @rosetn
This is an automated message from the Apache Git Service.
To respond to the message, please l
robinyqiu commented on pull request #12247:
URL: https://github.com/apache/beam/pull/12247#issuecomment-657880976
I moved `SqlOperators` to the `translation` sub-directory and changed its
content. For some reason GitHub does not recognize that. Instead it treats the
change as a file deleti
robinyqiu opened a new pull request #12247:
URL: https://github.com/apache/beam/pull/12247
After we switched to `BeamZetaSqlCalcRel`, the Java function implementation
is no longer needed. But the code stayed there without being used, so I am
removing them now, along with some other unused
apilloud commented on pull request #12238:
URL: https://github.com/apache/beam/pull/12238#issuecomment-657875503
Run Seed Job
This is an automated message from the Apache Git Service.
To respond to the message, please log on
robertwb opened a new pull request #12246:
URL: https://github.com/apache/beam/pull/12246
This recently came up on the users list, and there was no good documentation
to be found.
Thank you for your contribution! Follow this checklist to help us
incor
robertwb commented on pull request #12246:
URL: https://github.com/apache/beam/pull/12246#issuecomment-657873436
R: @rosetn
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
amaliujia commented on a change in pull request #12232:
URL: https://github.com/apache/beam/pull/12232#discussion_r453996803
##
File path:
sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/rel/BeamMatchRel.java
##
@@ -0,0 +1,380 @@
+/*
+ * Licensed
amaliujia commented on a change in pull request #12232:
URL: https://github.com/apache/beam/pull/12232#discussion_r453995446
##
File path:
sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/cep/CEPTypeName.java
##
@@ -0,0 +1,33 @@
+/*
+ * Licensed t
robinyqiu commented on a change in pull request #12174:
URL: https://github.com/apache/beam/pull/12174#discussion_r453995996
##
File path:
sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/meta/provider/bigquery/BeamBigQuerySqlDialect.java
##
@@ -156,7
robertwb commented on pull request #12245:
URL: https://github.com/apache/beam/pull/12245#issuecomment-657851028
R: @rosetn
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
robertwb opened a new pull request #12245:
URL: https://github.com/apache/beam/pull/12245
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly and easily:
- [ ] [**Choose
reviewer(s)**](https://b
aaltay merged pull request #12244:
URL: https://github.com/apache/beam/pull/12244
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 commented on pull request #1:
URL: https://github.com/apache/beam/pull/1#issuecomment-657842821
I agree with both of you. Keeping tests by default would allow easy testing.
(But is it free for all even if theirs forks are private?) And making it opt-in
sounds like least disr
aaltay commented on a change in pull request #12172:
URL: https://github.com/apache/beam/pull/12172#discussion_r453991538
##
File path: sdks/python/apache_beam/runners/worker/crossplatform_time.h
##
@@ -0,0 +1,55 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
kennknowles merged pull request #12216:
URL: https://github.com/apache/beam/pull/12216
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
kennknowles commented on pull request #12216:
URL: https://github.com/apache/beam/pull/12216#issuecomment-657837672
Ah, green.
This is an automated message from the Apache Git Service.
To respond to the message, please log on
kennknowles commented on pull request #12224:
URL: https://github.com/apache/beam/pull/12224#issuecomment-657837441
Finally green!
This is an automated message from the Apache Git Service.
To respond to the message, please lo
kennknowles merged pull request #12225:
URL: https://github.com/apache/beam/pull/12225
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
kennknowles commented on pull request #12225:
URL: https://github.com/apache/beam/pull/12225#issuecomment-657835783
Finally green!
This is an automated message from the Apache Git Service.
To respond to the message, please lo
ihji commented on pull request #12244:
URL: https://github.com/apache/beam/pull/12244#issuecomment-657826568
R: @aaltay
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
ihji opened a new pull request #12244:
URL: https://github.com/apache/beam/pull/12244
**Please** add a meaningful description for your change here
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly a
robinyqiu commented on a change in pull request #12174:
URL: https://github.com/apache/beam/pull/12174#discussion_r453958885
##
File path:
sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/TestInput.java
##
@@ -258,6 +259,14 @@
amaliujia commented on a change in pull request #12232:
URL: https://github.com/apache/beam/pull/12232#discussion_r453979800
##
File path:
sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/rel/BeamMatchRel.java
##
@@ -0,0 +1,380 @@
+/*
+ * Licensed
kennknowles commented on pull request #12224:
URL: https://github.com/apache/beam/pull/12224#issuecomment-657812019
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
kennknowles commented on pull request #12225:
URL: https://github.com/apache/beam/pull/12225#issuecomment-657808372
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
kennknowles opened a new pull request #12243:
URL: https://github.com/apache/beam/pull/12243
Successful runs have been seen at 90 seconds. The timeout only needs to catch
deadlocks and crashloops and should not be sensitive to performance in either
the underlying code or our test infra
ZijieSong946 commented on pull request #12174:
URL: https://github.com/apache/beam/pull/12174#issuecomment-657806602
Merge conflict solved.
This is an automated message from the Apache Git Service.
To respond to the message,
kennknowles commented on pull request #12216:
URL: https://github.com/apache/beam/pull/12216#issuecomment-657799504
Run Java PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, pleas
kennknowles commented on pull request #12216:
URL: https://github.com/apache/beam/pull/12216#issuecomment-657799415
Generating license report succeeded. The gradle scan shows two flakes:
- https://issues.apache.org/jira/browse/BEAM-9187
- https://issues.apache.org/jira/browse/BEA
saavannanavati commented on a change in pull request #12009:
URL: https://github.com/apache/beam/pull/12009#discussion_r453938415
##
File path: sdks/python/apache_beam/typehints/decorators.py
##
@@ -378,6 +378,56 @@ def has_simple_output_type(self):
self.output_types a
saavannanavati commented on a change in pull request #12009:
URL: https://github.com/apache/beam/pull/12009#discussion_r453938415
##
File path: sdks/python/apache_beam/typehints/decorators.py
##
@@ -378,6 +378,56 @@ def has_simple_output_type(self):
self.output_types a
kennknowles merged pull request #12227:
URL: https://github.com/apache/beam/pull/12227
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
robinyqiu merged pull request #12231:
URL: https://github.com/apache/beam/pull/12231
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
robinyqiu commented on pull request #12231:
URL: https://github.com/apache/beam/pull/12231#issuecomment-657787552
Merging now (tested internally)
This is an automated message from the Apache Git Service.
To respond to the me
robinyqiu commented on a change in pull request #12231:
URL: https://github.com/apache/beam/pull/12231#discussion_r453924913
##
File path:
sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/meta/provider/bigquery/BeamBigQuerySqlDialect.java
##
@@ -359,6
Hannah-Jiang commented on pull request #12216:
URL: https://github.com/apache/beam/pull/12216#issuecomment-657785349
> There was a failure to pull a license from another of the Beam modules
because the name was wrong. It was called `fn-execution:2.24.0-SNAPSHOT`. This
was already the case
kennknowles commented on pull request #12216:
URL: https://github.com/apache/beam/pull/12216#issuecomment-657780781
https://issues.apache.org/jira/browse/BEAM-9187 which said "sickbay" but is
clearly running
This is an autom
aaltay commented on pull request #11639:
URL: https://github.com/apache/beam/pull/11639#issuecomment-657779047
Run Java PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, please log
kennknowles commented on a change in pull request #11792:
URL: https://github.com/apache/beam/pull/11792#discussion_r453911740
##
File path:
runners/portability/java/src/main/java/org/apache/beam/runners/portability/testing/TestUniversalRunner.java
##
@@ -0,0 +1,119 @@
+/*
+ *
saavannanavati opened a new pull request #12242:
URL: https://github.com/apache/beam/pull/12242
This PR adds a load test for evaluating the performance of the runtime
typechecking system for the Python SDK. It works by comparing the performance
of a pipeline with the runtime typechecking f
santhh commented on a change in pull request #11566:
URL: https://github.com/apache/beam/pull/11566#discussion_r453907291
##
File path:
sdks/java/extensions/ml/src/main/java/org/apache/beam/sdk/extensions/ml/DLPDeidentifyText.java
##
@@ -0,0 +1,282 @@
+/*
+ * Licensed to the A
youngoli commented on pull request #12046:
URL: https://github.com/apache/beam/pull/12046#issuecomment-657760963
This PR is waiting for the dynamic splitting implementation to go in first,
since it's being used to test that. I'm working on that this week so it
shouldn't be much longer.
-
kennknowles commented on pull request #11792:
URL: https://github.com/apache/beam/pull/11792#issuecomment-657760388
Flakes in this run:
- https://issues.apache.org/jira/browse/BEAM-10474 failure in py
DisplayDataTest
And then I just introduced an actual compile error in my fi
kennknowles commented on pull request #12224:
URL: https://github.com/apache/beam/pull/12224#issuecomment-657744481
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
kennknowles commented on pull request #12225:
URL: https://github.com/apache/beam/pull/12225#issuecomment-657743373
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
robinyqiu merged pull request #12237:
URL: https://github.com/apache/beam/pull/12237
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
ffDante commented on pull request #4341:
URL: https://github.com/apache/beam/pull/4341#issuecomment-657742217
It seems like this is still not working with Dataflow Runner
This is an automated message from the Apache Git Servi
chamikaramj merged pull request #11919:
URL: https://github.com/apache/beam/pull/11919
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
kennknowles commented on pull request #12216:
URL: https://github.com/apache/beam/pull/12216#issuecomment-657737628
There was a failure to pull a license from another of the Beam modules
because the name was wrong. It was called `fn-execution:2.24.0-SNAPSHOT`. This
was already the case and
kennknowles commented on pull request #12216:
URL: https://github.com/apache/beam/pull/12216#issuecomment-657736077
Run Java PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, pleas
kennknowles commented on pull request #12216:
URL: https://github.com/apache/beam/pull/12216#issuecomment-657735289
Filed https://issues.apache.org/jira/browse/BEAM-10470 to track the flake in
https://ci-beam.apache.org/job/beam_PreCommit_Java_Commit/12326/testReport/junit/org.apache.beam.s
aaltay commented on pull request #12233:
URL: https://github.com/apache/beam/pull/12233#issuecomment-657728399
Do we expect workflows to run when there is a change to workflows?
This is an automated message from the Apache Gi
pabloem commented on a change in pull request #12151:
URL: https://github.com/apache/beam/pull/12151#discussion_r453845146
##
File path:
sdks/java/io/snowflake/src/main/java/org/apache/beam/sdk/io/snowflake/services/SnowflakeStreamingServiceImpl.java
##
@@ -0,0 +1,68 @@
+/*
+
lukecwik commented on pull request #12241:
URL: https://github.com/apache/beam/pull/12241#issuecomment-657716085
R: @boyuanzz
This is an automated message from the Apache Git Service.
To respond to the message, please log on
lukecwik opened a new pull request #12241:
URL: https://github.com/apache/beam/pull/12241
This will also be important to have this lock when splitting/getting
progress for window observing splittable DoFns.
Thank you for your contribution! Follow this
kennknowles commented on pull request #12224:
URL: https://github.com/apache/beam/pull/12224#issuecomment-657702113
Run Java PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, pleas
kennknowles commented on a change in pull request #12225:
URL: https://github.com/apache/beam/pull/12225#discussion_r453818020
##
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/io/LocalFileSystem.java
##
@@ -90,9 +91,14 @@
@Override
protected List match(Li
robinyqiu commented on a change in pull request #12237:
URL: https://github.com/apache/beam/pull/12237#discussion_r453821984
##
File path:
sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/pubsub/PubsubSchemaCapableIOProvider.java
##
@@ -116,11 +116,2
apilloud commented on pull request #12240:
URL: https://github.com/apache/beam/pull/12240#issuecomment-657697498
R: @lukecwik
This is an automated message from the Apache Git Service.
To respond to the message, please log on
apilloud opened a new pull request #12240:
URL: https://github.com/apache/beam/pull/12240
NaN values currently cause data corruption for Min() and Max(). Fix and add
tests.
Thank you for your contribution! Follow this checklist to help us
incorporate
kennknowles commented on pull request #12216:
URL: https://github.com/apache/beam/pull/12216#issuecomment-657694727
Rebasing to squash in fixup commit, then will merge.
This is an automated message from the Apache Git Service
emilymye commented on pull request #1:
URL: https://github.com/apache/beam/pull/1#issuecomment-657689639
I think my reasoning was that actions should be specifically opt-in, since
disabling them per-fork still requires a manual opt-out step. I guess I'm not
entirely sure why these
lazylynx opened a new pull request #12239:
URL: https://github.com/apache/beam/pull/12239
Made it possible for some tests to configure which python version to use
with gradle properties
Thank you for your contribution! Follow this checklist to help us
robertwb commented on pull request #11963:
URL: https://github.com/apache/beam/pull/11963#issuecomment-657688336
I've refactored this to use named tuples rather than rows. PTAL.
This is an automated message from the Apache Gi
damgad commented on pull request #12238:
URL: https://github.com/apache/beam/pull/12238#issuecomment-657685437
R: @aaltay
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
damgad commented on a change in pull request #12172:
URL: https://github.com/apache/beam/pull/12172#discussion_r453805179
##
File path: sdks/python/apache_beam/runners/worker/crossplatform_time.h
##
@@ -0,0 +1,55 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
damgad opened a new pull request #12238:
URL: https://github.com/apache/beam/pull/12238
Adding PostCommit jenkins job that builds Javadoc
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly and easily
TobKed commented on a change in pull request #12150:
URL: https://github.com/apache/beam/pull/12150#discussion_r453790997
##
File path: release/src/main/scripts/download_github_actions_artifacts.py
##
@@ -0,0 +1,234 @@
+#
+# Licensed to the Apache Software Foundation (ASF) unde
pabloem commented on pull request #12201:
URL: https://github.com/apache/beam/pull/12201#issuecomment-657662013
Run Python2_PVR_Flink PreCommit
This is an automated message from the Apache Git Service.
To respond to the messa
sclukas77 opened a new pull request #12237:
URL: https://github.com/apache/beam/pull/12237
Revamped SchemaCapableIOProvider to SchemaIOProvider in anticipation of
schema aware IOs, which, unlike SchemaCapableIOProviders, do not require a data
schema passed from the user. A requiresDataSche
lukecwik merged pull request #12226:
URL: https://github.com/apache/beam/pull/12226
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
1 - 100 of 169 matches
Mail list logo
