Default/tok

[pawanrawal]

1. Remove default tokenizer for all types. Return an error to user if he doesn't specify what tokenizer to use.
2. Rename DateTime tokenizer to Year tokenizer.

Fixes #1265

---

This change is 

[manishrjain]

---

Reviewed 9 of 9 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks broke.

---

schema/parse.go, line 165 at r1 (raw file):

		def := tok.Default(typ)
		// For string, datatime we don't have default tokenizer so user should specify one.
		if def == nil {

handle the smaller case first.

if def != nil {
return ...
}
return ...

---

Comments from Reviewable

[pawanrawal]

Review status: 3 of 10 files reviewed at latest revision, 1 unresolved discussion.

---

schema/parse.go, line 165 at r1 (raw file):

Previously, manishrjain (Manish R Jain) wrote…

handle the smaller case first.

if def != nil {
return ...
}
return ...

Noted, though we dont need it anymore because we removed default tokenizers.

---

Comments from Reviewable

[janardhan1993]

---

Review status: 3 of 10 files reviewed at latest revision, 1 unresolved discussion.

---

Comments from Reviewable

[pawanrawal]

Merged to master.