Skip to content

[BEAM-8932][Cleanup] Extract IncomingMessage and OutgoingMessage from PubsubClient - #10480

Closed
dpcollins-google wants to merge 3 commits into
apache:masterfrom
dpcollins-google:extract-client-messages
Closed

[BEAM-8932][Cleanup] Extract IncomingMessage and OutgoingMessage from PubsubClient#10480
dpcollins-google wants to merge 3 commits into
apache:masterfrom
dpcollins-google:extract-client-messages

Conversation

@dpcollins-google

@dpcollins-googledpcollins-google commented Dec 29, 2019

Copy link
Copy Markdown
Contributor

Diffbased on #10479

  • Choose reviewer(s) and mention them in a comment (R: @username).
  • Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Post-Commit Tests Status (on master branch)

LangSDKApexDataflowFlinkGearpumpSamzaSpark
GoBuild Status------Build Status------Build Status
JavaBuild StatusBuild StatusBuild StatusBuild Status
Build Status
Build Status
Build StatusBuild StatusBuild Status
Build Status
Build Status
PythonBuild Status
Build Status
Build Status
Build Status
---Build Status
Build Status
Build Status
Build Status
------Build Status
XLang---------Build Status---------

Pre-Commit Tests Status (on master branch)

---JavaPythonGoWebsite
Non-portableBuild StatusBuild Status
Build Status
Build StatusBuild Status
Portable---Build Status------

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@dpcollins-google

dpcollins-google commented Dec 29, 2019

Copy link
Copy Markdown
ContributorAuthor

+R: @chamikaramj

@aaltay

Copy link
Copy Markdown
Member

R: @chamikaramj could you find a person to review this PR?

@dpcollins-google
dpcollins-googleforce-pushed the extract-client-messages branch 4 times, most recently from 5f70308 to fd67defCompareJanuary 9, 2020 22:31
@dpcollins-google
dpcollins-googleforce-pushed the extract-client-messages branch 4 times, most recently from 55145bf to a26ac21CompareFebruary 8, 2020 17:33
@dpcollins-google
dpcollins-googleforce-pushed the extract-client-messages branch 3 times, most recently from 87a4c08 to 10c8cedCompareFebruary 11, 2020 19:57
@dpcollins-googledpcollins-google changed the title Extract IncomingMessage and OutgoingMessage from PubsubClient[BEAM-8932][Cleanup] Extract IncomingMessage and OutgoingMessage from PubsubClientFeb 20, 2020
@aaltay
aaltay requested a review from chamikaramjMarch 11, 2020 21:26
@stale

staleBot commented May 10, 2020

Copy link
Copy Markdown

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 request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@beam.apache.org list. Thank you for your contributions.

@stalestaleBot added the stale label May 10, 2020
@aaltay

Copy link
Copy Markdown
Member

I believe this PR is still active.

@stalestaleBot removed the stale label May 11, 2020
@stale

staleBot commented Jul 11, 2020

Copy link
Copy Markdown

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 request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@beam.apache.org list. Thank you for your contributions.

@stalestaleBot added the stale label Jul 11, 2020
@stale

staleBot commented Jul 18, 2020

Copy link
Copy Markdown

This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

@stalestaleBot closed this Jul 18, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@dpcollins-google@aaltay