Skip to content

[SPARK-23142][SS][DOCS] Added docs for continuous processing #20308

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from

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 <tathagata.das1565@gmail.com>

Closes #20308 from tdas/SPARK-23142.

(cherry picked from commit 4cd2ecc)
Signed-off-by: Tathagata Das <tathagata.das1565@gmail.com>
@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