Skip to content
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

Roll forward "[SPARK-23096][SS] Migrate rate source to V2" #20921

Closed
wants to merge 1 commit into from

Conversation

jose-torres
Copy link
Contributor

What changes were proposed in this pull request?

Roll forward c68ec4e (#20688)

There are two minor test changes required:

  • An error which used to be TreeNodeException[ArithmeticException] is no longer wrapped and is now just ArithmeticException.
  • The test framework was not setting the active Spark session for some reason.

How was this patch tested?

existing unit tests

@jose-torres
Copy link
Contributor Author

I rebased wrong. Let me fix the commit history.

@SparkQA
Copy link

SparkQA commented Mar 28, 2018

Test build #88668 has finished for PR 20921 at commit c2e6c92.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
  • class RateSourceSuite extends StreamTest
  • case class AdvanceRateManualClock(seconds: Long) extends AddData

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants