Re: [PR] Revert #30533: Automatically execute unbounded pipelines in streaming mode. [beam]

2024-04-09 Thread via GitHub


kennknowles commented on PR #30706:
URL: https://github.com/apache/beam/pull/30706#issuecomment-2045466784

   The tests are entirely internal to Google so there is no externally tracked 
failure. You would have to write a pipeline with unbounded data and then insist 
on running it in batch mode, which presumably not many people do.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Revert #30533: Automatically execute unbounded pipelines in streaming mode. [beam]

2024-04-09 Thread via GitHub


kennknowles commented on PR #30706:
URL: https://github.com/apache/beam/pull/30706#issuecomment-2045464303

   The description is not accurate - we have an underlying bug in dataflow and 
some erroneous tests. The original commit was a correct change. This is 
reverted while we fix our tests and service.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Revert #30533: Automatically execute unbounded pipelines in streaming mode. [beam]

2024-03-22 Thread via GitHub


damondouglas merged PR #30706:
URL: https://github.com/apache/beam/pull/30706


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Revert #30533: Automatically execute unbounded pipelines in streaming mode. [beam]

2024-03-22 Thread via GitHub


damccorm commented on PR #30706:
URL: https://github.com/apache/beam/pull/30706#issuecomment-2015315124

   My understanding is that this has caused some Dataflow tests to fail, but 
has it actually had negative impact on real pipelines? Is the problem here or 
is it with the tests themselves?
   
   FYI @robertwb 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Revert #30533: Automatically execute unbounded pipelines in streaming mode. [beam]

2024-03-22 Thread via GitHub


riteshghorse commented on PR #30706:
URL: https://github.com/apache/beam/pull/30706#issuecomment-2015280947

   Do we have any issue tracking this or failure run we can link?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Revert #30533: Automatically execute unbounded pipelines in streaming mode. [beam]

2024-03-21 Thread via GitHub


github-actions[bot] commented on PR #30706:
URL: https://github.com/apache/beam/pull/30706#issuecomment-2014007401

   Assigning reviewers. If you would like to opt out of this review, comment 
`assign to next reviewer`:
   
   R: @riteshghorse for label python.
   
   Available commands:
   - `stop reviewer notifications` - opt out of the automated review tooling
   - `remind me after tests pass` - tag the comment author after tests pass
   - `waiting on author` - shift the attention set back to the author (any 
comment or push by the author will return the attention set to the reviewers)
   
   The PR bot will only process comments in the main thread (not review 
comments).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] Revert #30533: Automatically execute unbounded pipelines in streaming mode. [beam]

2024-03-21 Thread via GitHub


damondouglas opened a new pull request, #30706:
URL: https://github.com/apache/beam/pull/30706

   This reverts commit 1c55117fc3349508edf8af07b1b361313c4b8a33.
   
   This commit has been associated with failures related to data sampling in 
Dataflow.
   
   
   
   Thank you for your contribution! Follow this checklist to help us 
incorporate your contribution quickly and easily:
   
- N/A internal: ~Mention the appropriate issue in your description (for 
example: `addresses #123`), if applicable. This will automatically add a link 
to the pull request in the issue. If you would like the issue to automatically 
close on merging the pull request, comment `fixes #` instead.~
   
   See the [Contributor Guide](https://beam.apache.org/contribute) for more 
tips on [how to make review process 
smoother](https://github.com/apache/beam/blob/master/CONTRIBUTING.md#make-the-reviewers-job-easier).
   
   To check the build health, please visit 
[https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md](https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md)
   
   GitHub Actions Tests Status (on master branch)
   

   [![Build python source distribution and 
wheels](https://github.com/apache/beam/workflows/Build%20python%20source%20distribution%20and%20wheels/badge.svg?branch=master=schedule)](https://github.com/apache/beam/actions?query=workflow%3A%22Build+python+source+distribution+and+wheels%22+branch%3Amaster+event%3Aschedule)
   [![Python 
tests](https://github.com/apache/beam/workflows/Python%20tests/badge.svg?branch=master=schedule)](https://github.com/apache/beam/actions?query=workflow%3A%22Python+Tests%22+branch%3Amaster+event%3Aschedule)
   [![Java 
tests](https://github.com/apache/beam/workflows/Java%20Tests/badge.svg?branch=master=schedule)](https://github.com/apache/beam/actions?query=workflow%3A%22Java+Tests%22+branch%3Amaster+event%3Aschedule)
   [![Go 
tests](https://github.com/apache/beam/workflows/Go%20tests/badge.svg?branch=master=schedule)](https://github.com/apache/beam/actions?query=workflow%3A%22Go+tests%22+branch%3Amaster+event%3Aschedule)
   
   See [CI.md](https://github.com/apache/beam/blob/master/CI.md) for more 
information about GitHub Actions CI or the [workflows 
README](https://github.com/apache/beam/blob/master/.github/workflows/README.md) 
to see a list of phrases to trigger workflows.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org