Skip to content

Add Blog Post on TestStream#46

Closed
tgroh wants to merge 6 commits intoapache:asf-sitefrom
tgroh:test_stream_blog_post
Closed

Add Blog Post on TestStream#46
tgroh wants to merge 6 commits intoapache:asf-sitefrom
tgroh:test_stream_blog_post

Conversation

@tgroh
Copy link
Member

@tgroh tgroh commented Oct 17, 2016

The post goes into details on the previous and expanded testing
infrastructure provided by the Beam SDK and provides examples of the
times at which elements arrive, and how that impacts the outputs
produced by a PipelineRunner.

The post goes into details on the previous and expanded testing
infrastructure provided by the Beam SDK and provides examples of the
times at which elements arrive, and how that impacts the outputs
produced by a PipelineRunner.
@tgroh tgroh force-pushed the test_stream_blog_post branch from ec8f638 to ef0c5ce Compare October 18, 2016 18:42
@tgroh
Copy link
Member Author

tgroh commented Oct 18, 2016

R: @francesperry

@tgroh
Copy link
Member Author

tgroh commented Oct 18, 2016

I haven't yet gotten everything to render locally, fwiw

Copy link
Member

@francesperry francesperry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few quick thoughts just on the markdown. Let me know when you've had a chance to check the rendering yourself and I'll do a more detailed review.

title: "Testing Unbounded Pipeliens in Apache Beam"
date: 2016-10-17 10:21:48 -0800
excerpt_separator: <!--more-->
categories: blog beam testing java
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't add all these. They make the URL weird. I'd just stick to 'blog'. Also filed BEAM-776 to figure out how to make this not ugly.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

(Picture me collapsing at the knees and cradling my head)

excerpt_separator: <!--more-->
categories: blog beam testing java
authors:
- tgroh
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to add yourself to _data/authors.yml

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

primitives provide a way for users to perform useful, powerful, and correct
computations in spite of these challenges.

We need comprehensive tests that cover crucial failure scenarios and corner
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps preface with "As users of Beam, we..." to clarify this isn't referring to testing Beam itself.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

unit tests for pipelines that may receive late data or trigger multiple times
has historically ranged from complex to not possible, as pipelines that read
from unbounded sources do not shut down without external intervention, while
Pipelines that read from bounded sources exclusively cannot test behavior with
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pipelines -> pipelines?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

that use custom triggers and handle out-of-order data could not be easily tested.

This blog post introduces our new framework for writing tests for pipelines that
handle delayed and out-of-order data in the context of the LeaderBoard/Mobile
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leaderboard pipeline from the Mobile Gaming example series.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@tgroh tgroh force-pushed the test_stream_blog_post branch 2 times, most recently from 00d1a25 to ebc54c0 Compare October 19, 2016 17:47
@tgroh
Copy link
Member Author

tgroh commented Oct 19, 2016

Generated static content. Fixed spelling of title.

I will need to update the post date before merge, including content regeneration.

@francesperry
Copy link
Member

Please don't regenerate content/ -- just leave that whole directory out and let the person doing the final merge do it. Also, see new shiny website instructions.

Also, you'll want to rename the file to the date you expect to publish. 10/20 seem reasonable?

@tgroh tgroh force-pushed the test_stream_blog_post branch from ebc54c0 to 883d4bc Compare October 19, 2016 20:39
@tgroh
Copy link
Member Author

tgroh commented Oct 19, 2016

Gone has the content.

Moved everything over.

@tgroh tgroh force-pushed the test_stream_blog_post branch from 883d4bc to 3170dd2 Compare October 19, 2016 20:41
@tgroh tgroh force-pushed the test_stream_blog_post branch 3 times, most recently from c5e369b to e4cdf42 Compare October 19, 2016 20:43
@francesperry
Copy link
Member

Looks good to me. Merging.

@asfgit asfgit closed this in fdd12fa Oct 20, 2016
robertwb pushed a commit to robertwb/incubator-beam that referenced this pull request Jun 5, 2018
melap pushed a commit to apache/beam that referenced this pull request Jun 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants