Release/fixes2

[pawanrawal]

This change is 

[ashwin95r]

---

Reviewed 7 of 7 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

---

cmd/dgraph/main.go, line 269 at r1 (raw file):

	if dgraph.Config.DebugMode {
		fmt.Printf("Received query: %+v\n", q)

fmt or x?

---

Comments from Reviewable

[janardhan1993]

---

Reviewed 6 of 7 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

---

cmd/dgraphloader/main.go, line 292 at r1 (raw file):

	// Lets call this so that badger is closed properly.
	dgraphClient.Close()

why not close in flush method ? after calling flush we wait for all mutations to finish ?

---

Comments from Reviewable

[pawanrawal]

Review status: 7 of 9 files reviewed at latest revision, 2 unresolved discussions.

---

cmd/dgraph/main.go, line 269 at r1 (raw file):

Previously, ashwin95r (Ashwin Ramesh) wrote…

fmt or x?

This is main.go, so fmt should be ok.

---

cmd/dgraphloader/main.go, line 292 at r1 (raw file):

Previously, janardhan1993 (Janardhan Reddy) wrote…

why not close in flush method ? after calling flush we wait for all mutations to finish ?

Flush is specific to batch. We need to close after we are done using dgraphClient which could be used for individual mutations (which need the kv) too.

---

Comments from Reviewable

[pawanrawal]

Merged