Skip to content

Conversation

@jagadeesanas2
Copy link
Contributor

What changes were proposed in this pull request?

The Maven module common/network-shuffle does not have a log4j configuration file for its test cases. So, added log4j.properties in the directory src/test/resources.

…shuffle]

@srowen
Copy link
Member

srowen commented Sep 15, 2016

OK, this is just a copy of another config file?

@SparkQA
Copy link

SparkQA commented Sep 15, 2016

Test build #65423 has finished for PR 15108 at commit f476755.

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

@jagadeesanas2
Copy link
Contributor Author

jagadeesanas2 commented Sep 15, 2016

yes, its some what similar to common/network-common

@srowen
Copy link
Member

srowen commented Sep 16, 2016

Merged to master

@asfgit asfgit closed this in b2e2726 Sep 16, 2016
wgtmac pushed a commit to wgtmac/spark that referenced this pull request Sep 19, 2016
## What changes were proposed in this pull request?

The Maven module `common/network-shuffle` does not have a log4j configuration file for its test cases. So, added `log4j.properties` in the directory `src/test/resources`.

…shuffle]

Author: Jagadeesan <[email protected]>

Closes apache#15108 from jagadeesanas2/SPARK-17543.
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