-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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
Update BasicOperationsSuite.scala #1084
Conversation
use the full package name for ManualClock
Can one of the admins verify this patch? |
@@ -22,7 +22,7 @@ import org.apache.spark.streaming.StreamingContext._ | |||
import org.apache.spark.rdd.{BlockRDD, RDD} | |||
import org.apache.spark.SparkContext._ | |||
|
|||
import util.ManualClock | |||
import org.apache.spark.streaming.util.ManualClock |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do u mind sorting the imports according to the coding style?
https://cwiki.apache.org/confluence/display/SPARK/Spark+Code+Style+Guide#SparkCodeStyleGuide-Imports
had modify the order of imports, pls see if it is ok, thanks @rxin |
Jenkins, test this please. |
This still does not match the import style. Please use the style here: https://cwiki.apache.org/confluence/display/SPARK/Spark+Code+Style+Guide#SparkCodeStyleGuide-Imports |
Merged build triggered. |
Merged build started. |
Merged build finished. |
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15976/ |
Compilation failed ... |
let me do a check |
closed |
* [CARMEL-6280] Data download query stuck in job commit phase * Add more.
use the full package name for ManualClock