DataType should be an enum

[JonathanGiles]

It seems that there is only a small number of valid dataTypes - string and binary (perhaps more, I haven't reviewed the docs exhaustively). It would be wise in my opinion to replace the String type with a custom enumeration, to prevent developer confusion over what the valid values are.