Skip to content

Latest commit

 

History

History
225 lines (163 loc) · 9.81 KB

CONTRIBUTING.md

File metadata and controls

225 lines (163 loc) · 9.81 KB

Contributing to Dgraph

Getting Started

Setting Up the Development Environment

Prerequisites

  • Install Git (may be already installed on your system, or available through your OS package manager)
  • Install Make (may be already installed on your system, or available through your OS package manager)
  • Install Docker and Docker Compose.
  • Install Go 1.13 or above.

Setup Dgraph from source repo

It's best to put the Dgraph repo somewhere in $GOPATH.

$ mkdir -p "$(go env GOPATH)/src/github.com/dgraph-io"
$ cd "$(go env GOPATH)/src/github.com/dgraph-io"
$ git clone https://github.com/dgraph-io/dgraph.git
$ cd ./dgraph
$ make install

This will put the source code in a Git repo under $GOPATH/src/github.com/dgraph-io/dgraph and compile the binaries to $GOPATH/bin.

Setup Badger from source repo

Dgraph source repo vendors its own version of Badger. If you are just working on Dgraph, you do not necessarily need to check out Badger from its own repo. However, if you want to contribute to Badger as well, you will need to check it out from its own repo.

$ go get -t -v github.com/dgraph-io/badger

This will put the source code in a Git repo under $GOPATH/src/github.com/dgraph-io/badger.

Protocol buffers

We use protocol buffers to serialize data between our server and the Go client and also for inter-worker communication. If you make any changes to the .proto files, you would have to recompile them.

Install the protoc compiler which is required for compiling proto files used for gRPC communication. Get protoc version 3.0.0 or above from GitHub releases page (look for the binary releases at the bottom, or compile from sources following the instructions).

We use gogo protobuf in Dgraph. To get the protocol buffer compiler plugin from gogo run

$ go get -u github.com/gogo/protobuf/protoc-gen-gofast

To compile the proto file using the protoc plugin and the gogo compiler plugin run the command make regenerate from within the directory containing the .proto files.

$ cd protos
$ make regenerate

This should generate the required .pb.go file.

Build Dgraph

You can build Dgraph using make dgraph or make install which add the version information to the binary.

  • make dgraph: Creates a dgraph binary at ./dgraph/dgraph
  • make install: Creates a dgraph binary at $GOPATH/bin/dgraph. You can add $GOPATH/bin to your $PATH.
$ make install
$ dgraph version
[Decoder]: Using assembly version of decoder

Dgraph version   : v1.1.1
Dgraph SHA-256   : 97326c9328aff93851290b12d846da81a7da5b843e97d7c63f5d79091b9063c1
Commit SHA-1     : 8994a57
Commit timestamp : 2019-12-16 18:24:50 -0800
Branch           : HEAD
Go version       : go1.13.5

For Dgraph official documentation, visit https://dgraph.io/docs/.
For discussions about Dgraph     , visit https://discuss.dgraph.io.

Licensed variously under the Apache Public License 2.0 and Dgraph Community License.
Copyright 2015-2023 Dgraph Labs, Inc.

Build Docker Image

make image

To build a test Docker image from source, use make image. This builds a Dgraph binary using make dgraph and creates a Docker image named dgraph/dgraph tagged as the current branch name. The image only contains the dgraph binary.

Example:

$ git rev-parse --abbrev-ref HEAD # current branch
main
$ make image
Successfully built c74d564d911f
Successfully tagged dgraph/dgraph:main
$ $ docker run --rm -it dgraph/dgraph:main dgraph version
[Decoder]: Using assembly version of decoder

Dgraph version   : v1.1.1-1-g5fa139a0e
Dgraph SHA-256   : 31f8c9324eb90a6f4659066937fcebc67bbca251c20b9da0461c2fd148187689
Commit SHA-1     : 5fa139a0e
Commit timestamp : 2019-12-16 20:52:06 -0800
Branch           : main
Go version       : go1.13.5

For Dgraph official documentation, visit https://dgraph.io/docs/.
For discussions about Dgraph     , visit https://discuss.dgraph.io.

Licensed variously under the Apache Public License 2.0 and Dgraph Community License.
Copyright 2015-2023 Dgraph Labs, Inc.

For release images, follow Doing a release. It creates Docker images that contains dgraph and badger commands.

Testing

Dgraph

  1. Change directory to t directory.

  2. If all packages need to be tested, run make test If only a specific package needs to be tested, run make test args="--pkg=desired_package_name"

    example 1: make test args="--pkg=tok" example 2: make test args="--pkg=tlstest/acl"

    The first example will run all the tests in the 'tok' directory (if there are any) The second one will run all the test in the acl subfolder of the tlstest directory. Note: running make test args="--pkg=tlstest" will return an error saying no packages found because all the tests in the tlstest package are in subdirectories of the package. So the subdirectories must be specified as shown in example 2.

Tests should be written in Go and use the Dgraph cluster set up in dgraph/docker-compose.yml whenever possible. If the functionality being tested requires a different cluster setup (e.g. different commandline options), the *_test.go files should be put in a separate directory that also contains a docker-compose.yml to set up the cluster as needed.

IMPORTANT: All containers should be labeled with cluster: test so they may be correctly restarted and cleaned up by the test script.

Badger

Run go test in the root folder.

$ go test ./...
ok      github.com/dgraph-io/badger     24.853s
ok      github.com/dgraph-io/badger/skl 0.027s
ok      github.com/dgraph-io/badger/table       0.478s
ok      github.com/dgraph-io/badger/y   0.004s

Contributing

Guidelines

Over years of writing big scalable systems, we are convinced that striving for simplicity wherever possible is the only way to build robust systems. This simplicity could be in design, could be in coding, or could be achieved by rewriting an entire module, that you may have painstakingly finished yesterday.

  • Pull requests are welcome, as long as you're willing to put in the effort to meet the guidelines. After you fork dgraph, create your pull request against our main branch
  • Contributors are required to execute our Individual Contributor License Agreement
  • Aim for clear, well written, maintainable code
  • Simple and minimal approach to features, like Go
  • New features must include passing unit tests, and integration tests when appropriate
  • Refactoring existing code now for better performance, better readability or better testability wins over adding a new feature
  • Don't add a function to a module that you don't use right now, or doesn't clearly enable a planned functionality
  • Don't ship a half done feature, which would require significant alterations to work fully
  • Avoid Technical debt like cancer
  • Leave the code cleaner than when you began

Code style

  • We're following Go Code Review
  • Use go fmt to format your code before committing
  • If you see any code which clearly violates the style guide, please fix it and send a pull request. No need to ask for permission
  • Avoid unnecessary vertical spaces. Use your judgment or follow the code review comments
  • Wrap your code and comments to 120 characters, unless doing so makes the code less legible

License Header

Every new source file must begin with a license header.

Most of Dgraph, Badger, and the Dgraph clients (dgo, dgraph-js, pydgraph and dgraph4j) are licensed under the Apache 2.0 license:

/*
 * Copyright 2016-2023 Dgraph Labs, Inc. and Contributors
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *    http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

Signed Commits

Signed commits help in verifying the authenticity of the contributor. We use signed commits in Dgraph, and we prefer it, though it's not compulsory to have signed commits. This is a recommended step for people who intend to contribute to Dgraph on a regular basis.

Follow instructions to generate and setup GPG keys for signing code commits on this Github Help page.