[IDE] cannot import src/main/scala & src/test/scala into eclipse as sour...

[ligangty]

...ce folder

When import the whole project into eclipse as maven project, found that the
src/main/scala & src/test/scala can not be set as source folder as default
behavior, so add a "add-source" goal in scala-maven-plugin to let this work.

[AmplabJenkins]

Can one of the admins verify this patch?

[srowen]

Hm, I wonder why? src/main/scala and src/test/scala should be the default locations picked up by the Maven compiler / scala plugins. You shouldn't have to configure anything else. Has anyone else reported this behavior? I'd be surprised if Eclipse has a particular problem since IJ (and Maven) don't.

[ligangty]

I'm not sure why it has this behavior, but the eclipse surely act as that. I've tested in Eclipse 3.8 and 4.3 with scala-plugin and scala-maven-plugin in both linux and windows platform, and got the same results. And after add this add-source goal, the src/main/scala will be imported as source folder which is reasonable.

[srowen]

Can anyone else using Eclipse comment? I know people have been using Spark in Eclipse successfully already.

[srowen]

ok to test

[SparkQA]

Test build #27396 has started for PR 4531 at commit 4e4db4c.

• This patch merges cleanly.

[SparkQA]

Test build #27396 has finished for PR 4531 at commit 4e4db4c.

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

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27396/
Test PASSed.

[srowen]

OK. Can't really hurt as the effect should just be to establish src/main/scala as a source root, which it already should be. This is small, but could you make a quick JIRA for this one and update the title with SPARK-xxxx [BUILD] ...?