Bazaar/Parser doesn't correctly escape some characters in TSV

[netj]

For example, carriage returns should also be escaped properly but not, hence causing troubles like HazyResearch/deepdive#523.

I think this part of the code needs more careful work to conform to Postgres' TSV format or some other stricter standard:

bazaar/parser/src/main/scala/com/clearcut/nlp/DocumentParser.scala

Line 98 in c09dce2

str.replace("\n", " ").replace("\t", " ")

[raphaelhoffmann]

Thanks, @netj! Since bazaar/parser is redundant and doesn't add much value, I'd favor removing it and migrating downstream applications to bazaar/pipe (or CoreNLP without a wrapper). We're also using JSON for serdes of NLP annotations to avoid charset and tsv issues.

[lanphan]

@raphaelhoffmann I got problem with Bazaar/parser setup.sh (see HazyResearch/deepdive#530), what should I do to replace bazaar/parser by bazaar/pipe like you said? Is it possible to apply it to current Deepdive now?