[#34] Update corenlp to 3.9.1 and fix some issues

databricks / spark-corenlp

Stanford CoreNLP wrapper for Apache Spark

GNU General Public License v3.0

422 stars 120 forks source link

[#34] Update corenlp to 3.9.1 and fix some issues #35

Closed mengxr closed 6 years ago

mengxr commented 6 years ago

Update corenlp version to 3.9.1
Update cleanxml implement to avoid an NPE bug
Enabled coref test and configure Travis to increase Java heap size

Closes #34

jkbradley commented 6 years ago

Related: I am hitting "java.lang.OutOfMemoryError: Java heap space" when running tests. I tried increasing the travis opts to -Xmx8g and then hit "java.lang.OutOfMemoryError: GC overhead limit exceeded"

mengxr commented 6 years ago

We disabled sudo and hence the container memory limit is only 4GB (https://docs.travis-ci.com/user/reference/overview/). The commit that passed Travis was just lucky (finished before getting killed). I marked sudo as required and we should get 7.5GB this time.