Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Mar 22, 2016

What changes were proposed in this pull request?

This PR moves flume back to Spark as per the discussion in the dev mail-list.

How was this patch tested?

Existing Jenkins tests.

@zsxwing
Copy link
Member Author

zsxwing commented Mar 22, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 22, 2016

Test build #53796 has finished for PR 11895 at commit 19ed34d.

  • This patch fails from timeout after a configured wait of 250m.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@SparkQA
Copy link

SparkQA commented Mar 22, 2016

Test build #53817 has finished for PR 11895 at commit 19ed34d.

  • This patch fails PySpark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@zsxwing
Copy link
Member Author

zsxwing commented Mar 23, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53863 has finished for PR 11895 at commit 4352499.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53842 has finished for PR 11895 at commit 4352499.

  • This patch fails from timeout after a configured wait of 250m.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@zsxwing zsxwing changed the title [SPARK-14073][Streaming]Move flume back to Spark [SPARK-14073][Streaming][test-maven]Move flume back to Spark Mar 23, 2016
@zsxwing
Copy link
Member Author

zsxwing commented Mar 23, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53880 has finished for PR 11895 at commit 4352499.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@zsxwing
Copy link
Member Author

zsxwing commented Mar 23, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53882 has finished for PR 11895 at commit 4352499.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53887 has finished for PR 11895 at commit 4352499.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@zsxwing
Copy link
Member Author

zsxwing commented Mar 23, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53896 has finished for PR 11895 at commit 4352499.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@zsxwing
Copy link
Member Author

zsxwing commented Mar 23, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53901 has finished for PR 11895 at commit 4352499.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@zsxwing
Copy link
Member Author

zsxwing commented Mar 23, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53941 has finished for PR 11895 at commit 4352499.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@zsxwing
Copy link
Member Author

zsxwing commented Mar 23, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53947 has finished for PR 11895 at commit 4352499.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@zsxwing
Copy link
Member Author

zsxwing commented Mar 23, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53953 has finished for PR 11895 at commit 4352499.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@zsxwing
Copy link
Member Author

zsxwing commented Mar 23, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53961 has finished for PR 11895 at commit 4352499.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaFlumeEventCount
    • class SparkSink extends AbstractSink with Logging with Configurable
    • class FlumeInputDStream[T: ClassTag](
    • class SparkFlumeEvent() extends Externalizable
    • class FlumeEventServer(receiver: FlumeReceiver) extends AvroSourceProtocol
    • class FlumeReceiver(
    • class CompressionChannelPipelineFactory extends ChannelPipelineFactory
    • class FlumeUtils(object):

@SparkQA
Copy link

SparkQA commented Mar 23, 2016

Test build #53970 has finished for PR 11895 at commit 7e18879.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Mar 24, 2016

Test build #53985 has finished for PR 11895 at commit d1e7435.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Member Author

Choose a reason for hiding this comment

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

I can reproduce this behavior by putting Spark to some folder containing "@" locally.

Copy link
Member Author

Choose a reason for hiding this comment

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

Here are the debug output I tried in Jenkins:

Running test.org.apache.spark.sql.JavaDataFrameSuite
raw:file:/home/jenkins/workspace/SparkPullRequestBuilder%402/sql/core/target/scala-2.11/test-classes/text-suite.txt
Find: /home/jenkins/workspace/SparkPullRequestBuilder@2/sql/core/target/scala-2.11/test-classes/text-suite.txt
raw:file:/home/jenkins/workspace/SparkPullRequestBuilder%402/sql/core/target/scala-2.11/test-classes/text-suite.txt
Find: /home/jenkins/workspace/SparkPullRequestBuilder@2/sql/core/target/scala-2.11/test-classes/text-suite.txt
raw:file:/home/jenkins/workspace/SparkPullRequestBuilder%402/sql/core/target/scala-2.11/test-classes/text-suite.txt
Find: /home/jenkins/workspace/SparkPullRequestBuilder@2/sql/core/target/scala-2.11/test-classes/text-suite.txt
raw:file:/home/jenkins/workspace/SparkPullRequestBuilder%402/sql/core/target/scala-2.11/test-classes/text-suite.txt
Find: /home/jenkins/workspace/SparkPullRequestBuilder@2/sql/core/target/scala-2.11/test-classes/text-suite.txt
Tests run: 17, Failures: 0, Errors: 2, Skipped: 0, Time elapsed: 7.628 sec <<< FAILURE! - in test.org.apache.spark.sql.JavaDataFrameSuite
testTextLoad(test.org.apache.spark.sql.JavaDataFrameSuite)  Time elapsed: 0.356 sec  <<< ERROR!
org.apache.spark.sql.AnalysisException: Path does not exist: file:/home/jenkins/workspace/SparkPullRequestBuilder%402/sql/core/target/scala-2.11/test-classes/text-suite.txt;
    at test.org.apache.spark.sql.JavaDataFrameSuite.testTextLoad(JavaDataFrameSuite.java:349)

testGenericLoad(test.org.apache.spark.sql.JavaDataFrameSuite)  Time elapsed: 0.322 sec  <<< ERROR!
org.apache.spark.sql.AnalysisException: Path does not exist: file:/home/jenkins/workspace/SparkPullRequestBuilder%402/sql/core/target/scala-2.11/test-classes/text-suite.txt;
    at test.org.apache.spark.sql.JavaDataFrameSuite.testGenericLoad(JavaDataFrameSuite.java:311)

Running test.org.apache.spark.sql.JavaRowSuite
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0 sec - in test.org.apache.spark.sql.JavaRowSuite

Results :

Tests in error: 
  JavaDataFrameSuite.testGenericLoad:311 ? Analysis Path does not exist: file:/h...
  JavaDataFrameSuite.testTextLoad:349 ? Analysis Path does not exist: file:/home...

This is the line returned by Thread.currentThread().getContextClassLoader().getResource("text-suite.txt").toString()

file:/home/jenkins/workspace/SparkPullRequestBuilder%402/sql/core/target/scala-2.11/test-classes/text-suite.txt

@SparkQA
Copy link

SparkQA commented Mar 24, 2016

Test build #54007 has finished for PR 11895 at commit 14d859d.

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

@SparkQA
Copy link

SparkQA commented Mar 24, 2016

Test build #54008 has finished for PR 11895 at commit f5d5976.

  • This patch fails from timeout after a configured wait of 250m.
  • This patch merges cleanly.
  • This patch adds no public classes.

@zsxwing
Copy link
Member Author

zsxwing commented Mar 24, 2016

retest this please

@zsxwing
Copy link
Member Author

zsxwing commented Mar 24, 2016

Test build #54054 has started for PR 11895 at commit f5d5976.

Hung by DataFrameReaderWriterSuite

retest this please

@zsxwing
Copy link
Member Author

zsxwing commented Mar 24, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 24, 2016

Test build #54054 has finished for PR 11895 at commit f5d5976.

  • This patch fails from timeout after a configured wait of 250m.
  • This patch merges cleanly.
  • This patch adds no public classes.

@zsxwing
Copy link
Member Author

zsxwing commented Mar 24, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 25, 2016

Test build #54106 has finished for PR 11895 at commit f5d5976.

  • This patch fails from timeout after a configured wait of 250m.
  • This patch merges cleanly.
  • This patch adds no public classes.

@zsxwing
Copy link
Member Author

zsxwing commented Mar 25, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 25, 2016

Test build #54140 has finished for PR 11895 at commit f5d5976.

  • This patch fails from timeout after a configured wait of 250m.
  • This patch merges cleanly.
  • This patch adds no public classes.

@vanzin
Copy link
Contributor

vanzin commented Mar 25, 2016

LGTM once we fix the maven test issues. Thanks for taking care of this!

@zsxwing
Copy link
Member Author

zsxwing commented Mar 25, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 25, 2016

Test build #54208 has finished for PR 11895 at commit f5d5976.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@zsxwing
Copy link
Member Author

zsxwing commented Mar 25, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Mar 25, 2016

Test build #54214 has finished for PR 11895 at commit f5d5976.

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

@zsxwing
Copy link
Member Author

zsxwing commented Mar 25, 2016

Finally passed. Ping @rxin to take a look.

@rxin
Copy link
Contributor

rxin commented Mar 26, 2016

Thanks - merging in master.

@asfgit asfgit closed this in 24587ce Mar 26, 2016
@zsxwing zsxwing deleted the move-flume-back branch March 26, 2016 01:26
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.

4 participants