Change dataType on Representation into binary

[RubenVerborgh]

The dataType field would be highly redundant, since we also have the content type on the metadata.

What we really need to know, is whether we have a stream of strings or objects, which I propose to indicate with a binary field. Whatever subtype of non-binary it is, will be disclosed in the content type anyway.

[joachimvh]

I think this might also be a point to consider how much added value dataType has. Currently it is equal to contentType === INTERNAL_QUADS (and actually also equal to data.readableObjectMode, which unfortunately can only be checked starting from Node 12).

[RubenVerborgh]

I think this might also be a point to consider how much added value dataType has. Currently it is equal to contentType === INTERNAL_QUADS (and actually also equal to data.readableObjectMode, which unfortunately can only be checked starting from Node 12).

Exactly. But:

• If we use Use readable-stream library to have consistent streams #55, we have that functionality
• Still might be easier to just set the boolean flag and be in control ourselves