HazyResearch / mindbender

Tools for iterative knowledge base development with DeepDive
116 stars 32 forks source link

Longer suffixes split #64

Closed ThomasPalomares closed 8 years ago

ThomasPalomares commented 8 years ago

Comments about heap size issue for Elastic Search added in the file search/mindbender-search, not sure it's the best place though.

netj commented 8 years ago

After reflecting the comments, can you squash the numerous commits into a single one please? Thanks!

ThomasPalomares commented 8 years ago

@netj file modified to reflect the comments and commits squashed !

netj commented 8 years ago

Thanks for making the changes. Just two small suggestions. You don't need to amend the commit into one this time.

netj commented 8 years ago

LGTM Thanks for making all the changes!