Skip to content

Conversation

@tdas
Copy link
Contributor

@tdas tdas commented Jan 18, 2018

What changes were proposed in this pull request?

Added documentation for continuous processing. Modified two locations.

  • Modified the overview to have a mention of Continuous Processing.
  • Added a new section on Continuous Processing at the end.

image
image

How was this patch tested?

N/A

@SparkQA
Copy link

SparkQA commented Jan 18, 2018

Test build #86316 has finished for PR 20308 at commit 43f2399.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jan 18, 2018

Test build #86320 has finished for PR 20308 at commit bc13ec4.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@tdas
Copy link
Contributor Author

tdas commented Jan 18, 2018

@zsxwing @jose-torres please take a look.

@jose-torres
Copy link
Contributor

Looks very good.

Is the intent to swap in console examples once #20311 is submitted?

@tdas
Copy link
Contributor Author

tdas commented Jan 18, 2018

@jose-torres on my second thought, not really. its good to have a kafka example to make it clear that this works with kafka. while the right way would be do a console example here and update the kafka integration guide with a kafka example, I am inclined to punt that for the future.

@jose-torres
Copy link
Contributor

Works for me. The ability to just copy-paste and have it work doesn't seem necessary for an experimental feature.

LGTM

@SparkQA
Copy link

SparkQA commented Jan 18, 2018

Test build #86358 has finished for PR 20308 at commit 851483e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

asfgit pushed a commit that referenced this pull request Jan 19, 2018
## What changes were proposed in this pull request?

Added documentation for continuous processing. Modified two locations.
- Modified the overview to have a mention of Continuous Processing.
- Added a new section on Continuous Processing at the end.

![image](https://user-images.githubusercontent.com/663212/35083551-a3dd23f6-fbd4-11e7-9e7e-90866f131ca9.png)
![image](https://user-images.githubusercontent.com/663212/35083618-d844027c-fbd4-11e7-9fde-75992cc517bd.png)

## How was this patch tested?
N/A

Author: Tathagata Das <[email protected]>

Closes #20308 from tdas/SPARK-23142.

(cherry picked from commit 4cd2ecc)
Signed-off-by: Tathagata Das <[email protected]>
@asfgit asfgit closed this in 4cd2ecc Jan 19, 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.

3 participants