[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642181079 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] ibzib commented on pull request #11949: Simplify Python on Flink runner instructions.

2020-06-10 Thread GitBox
ibzib commented on pull request #11949: URL: https://github.com/apache/beam/pull/11949#issuecomment-642180751 > Do you think it's worth moving the version compatibility table lower in case people want to see what changes when they toggle the tab? It's kind of out of place here anyway.

[GitHub] [beam] amaliujia commented on pull request #11971: [BEAM-10230] @Ignore: BYTES works with LIKE.

2020-06-10 Thread GitBox
amaliujia commented on pull request #11971: URL: https://github.com/apache/beam/pull/11971#issuecomment-642176244 R: @robinyqiu This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [beam] amaliujia opened a new pull request #11971: [BEAM-10230] @Ignore: BYTES works with LIKE.

2020-06-10 Thread GitBox
amaliujia opened a new pull request #11971: URL: https://github.com/apache/beam/pull/11971 Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily: - [ ] [**Choose

[GitHub] [beam] udim commented on pull request #11966: [BEAM-10217] CALL_FUNCTION and CALL_METHOD fixes

2020-06-10 Thread GitBox
udim commented on pull request #11966: URL: https://github.com/apache/beam/pull/11966#issuecomment-642175471 > Seeing a failure in PortableRunnerTestWithExternalEnv.test_error_traceback_includes_user_code, not sure if related. https://issues.apache.org/jira/browse/BEAM-8878

[GitHub] [beam] robertwb commented on pull request #11949: Simplify Python on Flink runner instructions.

2020-06-10 Thread GitBox
robertwb commented on pull request #11949: URL: https://github.com/apache/beam/pull/11949#issuecomment-642170460 Clarified the tabs selection. (Do you think it's worth moving the version compatibility table lower in case people want to see what changes when they toggle the tab? It's kind

[GitHub] [beam] rehmanmuradali commented on a change in pull request #11924: [BEAM-8543] Dataflow streaming timers are not strictly time ordered when set earlier mid-bundle

2020-06-10 Thread GitBox
rehmanmuradali commented on a change in pull request #11924: URL: https://github.com/apache/beam/pull/11924#discussion_r438307621 ## File path: runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/StreamingModeExecutionContext.java

[GitHub] [beam] rehmanmuradali commented on a change in pull request #11924: [BEAM-8543] Dataflow streaming timers are not strictly time ordered when set earlier mid-bundle

2020-06-10 Thread GitBox
rehmanmuradali commented on a change in pull request #11924: URL: https://github.com/apache/beam/pull/11924#discussion_r438307437 ## File path: sdks/java/core/src/test/java/org/apache/beam/sdk/transforms/ParDoTest.java ## @@ -4040,7 +4043,8 @@ public void onTimer(

[GitHub] [beam] rehmanmuradali commented on a change in pull request #11924: [BEAM-8543] Dataflow streaming timers are not strictly time ordered when set earlier mid-bundle

2020-06-10 Thread GitBox
rehmanmuradali commented on a change in pull request #11924: URL: https://github.com/apache/beam/pull/11924#discussion_r438307281 ## File path: sdks/java/core/src/test/java/org/apache/beam/sdk/transforms/ParDoTest.java ## @@ -3938,10 +3939,11 @@ public void

[GitHub] [beam] rehmanmuradali commented on a change in pull request #11924: [BEAM-8543] Dataflow streaming timers are not strictly time ordered when set earlier mid-bundle

2020-06-10 Thread GitBox
rehmanmuradali commented on a change in pull request #11924: URL: https://github.com/apache/beam/pull/11924#discussion_r438306969 ## File path: runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/StreamingModeExecutionContext.java

[GitHub] [beam] tvalentyn commented on pull request #11966: [BEAM-10217] CALL_FUNCTION and CALL_METHOD fixes

2020-06-10 Thread GitBox
tvalentyn commented on pull request #11966: URL: https://github.com/apache/beam/pull/11966#issuecomment-642164503 Seeing a failure in PortableRunnerTestWithExternalEnv.test_error_traceback_includes_user_code, not sure if related. The change LGTM and verified that it addresses

[GitHub] [beam] robertwb commented on a change in pull request #11765: [BEAM-9322] Remove passthrough_pcollection_output_ids and force_generated_pcollection_output_ids flags

2020-06-10 Thread GitBox
robertwb commented on a change in pull request #11765: URL: https://github.com/apache/beam/pull/11765#discussion_r438299971 ## File path: sdks/python/apache_beam/transforms/ptransform.py ## @@ -270,11 +256,19 @@ def get_named_nested_pvalues(pvalueish): tagged_values =

[GitHub] [beam] amaliujia commented on pull request #11967: [BEAM-9992] | use Sets transform in BeamSQL

2020-06-10 Thread GitBox
amaliujia commented on pull request #11967: URL: https://github.com/apache/beam/pull/11967#issuecomment-642159934 I will also try to run this PR against internal test suite since it is a big change. This is an automated

[GitHub] [beam] robertwb commented on a change in pull request #11894: [BEAM-7074] Enable test_pardo_timers_clear for fn_runner

2020-06-10 Thread GitBox
robertwb commented on a change in pull request #11894: URL: https://github.com/apache/beam/pull/11894#discussion_r438299065 ## File path: sdks/python/apache_beam/runners/portability/fn_api_runner/fn_runner_test.py ## @@ -377,11 +377,6 @@ def process_timer(

[GitHub] [beam] robertwb commented on pull request #11838: [BEAM-9322] Modify the TestStream to output a dict when no output_tags are specified

2020-06-10 Thread GitBox
robertwb commented on pull request #11838: URL: https://github.com/apache/beam/pull/11838#issuecomment-642155990 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [beam] pabloem commented on pull request #11969: Remove a hack used to retrieve schematized data from HL7v2 messages, …

2020-06-10 Thread GitBox
pabloem commented on pull request #11969: URL: https://github.com/apache/beam/pull/11969#issuecomment-642155193 retest this please This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [beam] robertwb commented on a change in pull request #11916: [BEAM-10189] Add ValueState user state to python sdk

2020-06-10 Thread GitBox
robertwb commented on a change in pull request #11916: URL: https://github.com/apache/beam/pull/11916#discussion_r438296283 ## File path: sdks/python/apache_beam/transforms/userstate_test.py ## @@ -452,6 +459,40 @@ def clear_values(self,

[GitHub] [beam] robertwb commented on a change in pull request #11943: [BEAM-10209] Add without_defaults to Mean combiner in Python

2020-06-10 Thread GitBox
robertwb commented on a change in pull request #11943: URL: https://github.com/apache/beam/pull/11943#discussion_r438291777 ## File path: sdks/python/apache_beam/transforms/combiners_test.py ## @@ -105,6 +109,16 @@ def test_builtin_combines(self):

[GitHub] [beam] purbanow edited a comment on pull request #11794: [BEAM-9894] Add batch SnowflakeIO.Write to Java SDK

2020-06-10 Thread GitBox
purbanow edited a comment on pull request #11794: URL: https://github.com/apache/beam/pull/11794#issuecomment-642150480 Hey @RyanSkraba , We answered for all your comments. Will you find a moment to check again ? This is

[GitHub] [beam] purbanow edited a comment on pull request #11794: [BEAM-9894] Add batch SnowflakeIO.Write to Java SDK

2020-06-10 Thread GitBox
purbanow edited a comment on pull request #11794: URL: https://github.com/apache/beam/pull/11794#issuecomment-642150480 Hey @RyanSkraba , We answered fo all your comments. Will you find a moment to check again ? This is an

[GitHub] [beam] purbanow commented on pull request #11794: [BEAM-9894] Add batch SnowflakeIO.Write to Java SDK

2020-06-10 Thread GitBox
purbanow commented on pull request #11794: URL: https://github.com/apache/beam/pull/11794#issuecomment-642150480 Hey @RyanSkraba , will you find a moment to check again ? This is an automated message from the Apache Git

[GitHub] [beam] kkucharc commented on a change in pull request #11794: [BEAM-9894] Add batch SnowflakeIO.Write to Java SDK

2020-06-10 Thread GitBox
kkucharc commented on a change in pull request #11794: URL: https://github.com/apache/beam/pull/11794#discussion_r438289586 ## File path: sdks/java/io/snowflake/src/main/java/org/apache/beam/sdk/io/snowflake/SnowflakeIO.java ## @@ -447,6 +494,346 @@ public void

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642144427 Run Dataflow Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642144324 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642144102 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] tvalentyn merged pull request #11962: [BEAM-10227] Switches typing version modifier to python_full_version.

2020-06-10 Thread GitBox
tvalentyn merged pull request #11962: URL: https://github.com/apache/beam/pull/11962 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

[GitHub] [beam] ibzib merged pull request #11961: [BEAM-10225] Add log message when starting job server

2020-06-10 Thread GitBox
ibzib merged pull request #11961: URL: https://github.com/apache/beam/pull/11961 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

[GitHub] [beam] tvalentyn commented on a change in pull request #11970: [BEAM-7672] add cp37 wheel to acceptableWhlSpecs

2020-06-10 Thread GitBox
tvalentyn commented on a change in pull request #11970: URL: https://github.com/apache/beam/pull/11970#discussion_r438270807 ## File path: sdks/python/container/boot.go ## @@ -38,7 +38,7 @@ import ( ) var ( - acceptableWhlSpecs =

[GitHub] [beam] tvalentyn commented on pull request #11970: [BEAM-7672] add cp37 wheel to acceptableWhlSpecs

2020-06-10 Thread GitBox
tvalentyn commented on pull request #11970: URL: https://github.com/apache/beam/pull/11970#issuecomment-642131353 Run Python DockerBuild PreCommit This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] tvalentyn commented on pull request #11970: [BEAM-7672] add cp37 wheel to acceptableWhlSpecs

2020-06-10 Thread GitBox
tvalentyn commented on pull request #11970: URL: https://github.com/apache/beam/pull/11970#issuecomment-642131178 retest this please This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [beam] tvalentyn commented on pull request #11970: [BEAM-7672] add cp37 wheel to acceptableWhlSpecs

2020-06-10 Thread GitBox
tvalentyn commented on pull request #11970: URL: https://github.com/apache/beam/pull/11970#issuecomment-642130988 I think Jenkins hooks currently require a comment from a committer to get triggered. This is an automated

[GitHub] [beam] iemejia merged pull request #11909: [BEAM-8134] Grafana dashboards for Nexmark tests

2020-06-10 Thread GitBox
iemejia merged pull request #11909: URL: https://github.com/apache/beam/pull/11909 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

[GitHub] [beam] kamilwu removed a comment on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu removed a comment on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642116959 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [beam] kennknowles merged pull request #11820: [BEAM-10093] ZetaSql Nexmark variant

2020-06-10 Thread GitBox
kennknowles merged pull request #11820: URL: https://github.com/apache/beam/pull/11820 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

[GitHub] [beam] kennknowles commented on pull request #11820: [BEAM-10093] ZetaSql Nexmark variant

2020-06-10 Thread GitBox
kennknowles commented on pull request #11820: URL: https://github.com/apache/beam/pull/11820#issuecomment-642121612 Thanks for the review! Definitely made this simpler, better, more focused. This is an automated message from

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642120033 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642119197 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642119592 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642116959 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] tysonjh commented on pull request #11960: [BEAM-9999] Remove Gearpump runner.

2020-06-10 Thread GitBox
tysonjh commented on pull request #11960: URL: https://github.com/apache/beam/pull/11960#issuecomment-642101579 > Looks like you have caught all the places. LGTM Thanks Maximilian! This is an automated message from

[GitHub] [beam] tysonjh commented on pull request #11960: [BEAM-9999] Remove Gearpump runner.

2020-06-10 Thread GitBox
tysonjh commented on pull request #11960: URL: https://github.com/apache/beam/pull/11960#issuecomment-642101341 R: @aaltay This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [beam] aromanenko-dev commented on pull request #11396: [BEAM-9742] Add Configurable FluentBackoff to JdbcIO Write

2020-06-10 Thread GitBox
aromanenko-dev commented on pull request #11396: URL: https://github.com/apache/beam/pull/11396#issuecomment-642096389 retest this please This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642093374 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642081794 Run Seed Job This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [beam] henryken commented on pull request #11944: [BEAM-10210] Beam Katas for Kotlin Blog Post

2020-06-10 Thread GitBox
henryken commented on pull request #11944: URL: https://github.com/apache/beam/pull/11944#issuecomment-642079721 That looks good to me! This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642072375 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] rionmonster commented on pull request #11944: [BEAM-10210] Beam Katas for Kotlin Blog Post

2020-06-10 Thread GitBox
rionmonster commented on pull request #11944: URL: https://github.com/apache/beam/pull/11944#issuecomment-642066925 @henryken Yup, I tested through yesterday evening and things look as expected on my local environment after building via Hugo:

[GitHub] [beam] lazylynx commented on pull request #11970: [BEAM-7672] add cp37 wheel to acceptableWhlSpecs

2020-06-10 Thread GitBox
lazylynx commented on pull request #11970: URL: https://github.com/apache/beam/pull/11970#issuecomment-642065184 `beam_PreCommit_Python_DockerBuild` is not working now? This is an automated message from the Apache Git

[GitHub] [beam] lazylynx commented on pull request #11970: [BEAM-7672] add cp37 wheel to acceptableWhlSpecs

2020-06-10 Thread GitBox
lazylynx commented on pull request #11970: URL: https://github.com/apache/beam/pull/11970#issuecomment-642064890 R: @tvalentyn This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [beam] lazylynx opened a new pull request #11970: [BEAM-7672] add cp37 wheel to acceptableWhlSpecs

2020-06-10 Thread GitBox
lazylynx opened a new pull request #11970: URL: https://github.com/apache/beam/pull/11970 Accept Py3 wheels in SDK harness container. Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642060169 Run Seed Job This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [beam] tysonjh edited a comment on pull request #11960: [BEAM-9999] Remove Gearpump runner.

2020-06-10 Thread GitBox
tysonjh edited a comment on pull request #11960: URL: https://github.com/apache/beam/pull/11960#issuecomment-642048043 R: @mxm Are you able to review this for me please? This is an automated message from the Apache

[GitHub] [beam] tysonjh commented on pull request #11960: [BEAM-9999] Remove Gearpump runner.

2020-06-10 Thread GitBox
tysonjh commented on pull request #11960: URL: https://github.com/apache/beam/pull/11960#issuecomment-642048043 R: @mxm Are you able to review this for me? This is an automated message from the Apache Git Service. To

[GitHub] [beam] lastomato opened a new pull request #11969: Remove a hack used to retrieve schematized data from HL7v2 messages, …

2020-06-10 Thread GitBox
lastomato opened a new pull request #11969: URL: https://github.com/apache/beam/pull/11969 …now that the schamatized parser is available publicly. Also removed a few methods that are not used anywhere. R: @pabloem @jaketf Thank you for your

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642037822 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu removed a comment on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu removed a comment on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-641927471 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [beam] kamilwu removed a comment on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu removed a comment on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642025383 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642025383 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-642025600 Run Seed Job This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [beam] TobKed opened a new pull request #11968: [BEAM-10185] Build python wheels on GitHub Actions for Windows [dependent on BEAM-10184]

2020-06-10 Thread GitBox
TobKed opened a new pull request #11968: URL: https://github.com/apache/beam/pull/11968 dependent on https://github.com/apache/beam/pull/11877 Subtask of [BEAM-9388](https://issues.apache.org/jira/browse/BEAM-9388) Example GitHub Action runs available on my fork:

[GitHub] [beam] TobKed edited a comment on pull request #11877: [BEAM-10184] Build python wheels on GitHub Actions for Linux/MacOS

2020-06-10 Thread GitBox
TobKed edited a comment on pull request #11877: URL: https://github.com/apache/beam/pull/11877#issuecomment-641886596 @aaltay thank you for review. Answers for your questions: > on pull_request: > Can we trigger this with a phrase? And/Or limit it to python changes? I

[GitHub] [beam] henryken commented on pull request #11944: [BEAM-10210] Beam Katas for Kotlin Blog Post

2020-06-10 Thread GitBox
henryken commented on pull request #11944: URL: https://github.com/apache/beam/pull/11944#issuecomment-641935550 Have you tested the blog post on your local? There should be nothing extra you need to do. The build process should take care of it once it gets merged. Having

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-641927471 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu commented on pull request #11909: [BEAM-8134] Grafana dashboards for Nexmark tests

2020-06-10 Thread GitBox
kamilwu commented on pull request #11909: URL: https://github.com/apache/beam/pull/11909#issuecomment-641926095 FYI: https://github.com/apache/beam/pull/11956 This is an automated message from the Apache Git Service. To

[GitHub] [beam] kamilwu commented on pull request #11909: [BEAM-8134] Grafana dashboards for Nexmark tests

2020-06-10 Thread GitBox
kamilwu commented on pull request #11909: URL: https://github.com/apache/beam/pull/11909#issuecomment-641922547 > We should definitely have shorter views by default but stored data should not disapear if there are no technical constraints that force us to do so Ok, makes sense to

[GitHub] [beam] TobKed commented on pull request #11877: [BEAM-10184] Build python wheels on GitHub Actions for Linux/MacOS

2020-06-10 Thread GitBox
TobKed commented on pull request #11877: URL: https://github.com/apache/beam/pull/11877#issuecomment-641886596 @aaltay thank you for review. Answers for your questions: > on pull_request: > Can we trigger this with a phrase? And/Or limit it to python changes? I added

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-641867173 Run Spark Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-641866913 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-641865785 Run Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu removed a comment on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu removed a comment on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-641865155 Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-641865155 Direct Runner Nexmark Tests This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-641864568 Run Java TextIO Performance Test This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] aromanenko-dev commented on pull request #11396: [BEAM-9742] Add Configurable FluentBackoff to JdbcIO Write

2020-06-10 Thread GitBox
aromanenko-dev commented on pull request #11396: URL: https://github.com/apache/beam/pull/11396#issuecomment-641853251 Run Java PreCommit This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [beam] kamilwu commented on pull request #11956: [BEAM-8133] Publishing results of Nexmark tests to InfluxDB

2020-06-10 Thread GitBox
kamilwu commented on pull request #11956: URL: https://github.com/apache/beam/pull/11956#issuecomment-641851529 Run Seed Job This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [beam] nielm commented on pull request #11570: [BEAM-10047] Merge the stages 'Gather and Sort' and 'Create Batches'

2020-06-10 Thread GitBox
nielm commented on pull request #11570: URL: https://github.com/apache/beam/pull/11570#issuecomment-641849548 Retest this please This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [beam] darshanj opened a new pull request #11967: [BEAM-9992] | use Sets transform in BeamSQL

2020-06-10 Thread GitBox
darshanj opened a new pull request #11967: URL: https://github.com/apache/beam/pull/11967 Use Sets transforms from BeamSQL code Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[GitHub] [beam] darshanj commented on pull request #11967: [BEAM-9992] | use Sets transform in BeamSQL

2020-06-10 Thread GitBox
darshanj commented on pull request #11967: URL: https://github.com/apache/beam/pull/11967#issuecomment-641748346 R: @amaliujia This is an automated message from the Apache Git Service. To respond to the message, please log

<    1   2