forked from mesos/spark
-
Notifications
You must be signed in to change notification settings - Fork 0
Contributing to Spark
mateiz edited this page Jul 13, 2012
·
6 revisions
The Spark team welcomes contributions in the form of GitHub pull requests. Here are a few tips to get your contribution in:
- Break your work into small, single-purpose patches if possible. It's much harder to merge in a large change with a lot of disjoint features.
- Submit the patch as a GitHub pull request. For a tutorial, see the GitHub guides on forking a repo and sending a pull request.
- Follow the style of the existing codebase. Specifically, we use [[standard Scala
style guide|http://docs.scala-lang.org/style/]], but with the following changes:
- Maximum line length of 100 characters.
- Always import packages using absolute paths (e.g.
scala.collection.Mapinstead ofcollection.Map). - No "infix" syntax for methods other than operators. For example, don't write
table containsKey myKey; replace it withtable.containsKey(myKey).
- Add unit tests to your new code. We use ScalaTest for
testing. Just add a new Suite in
core/src/test, or methods to an existing Suite.
If you'd like to report a bug but don't have time to fix it, you can still post it to our issues page. Also, feel free to email the mailing list.