mcvsubbu commented on issue #4993: Support Text column type in Pinot (both offline and realtime) URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-585489411 Other general points discussied: * Disable TEXT_MATCH until we have implemented reload -- otherwise, adding a text index on a column (our primary use case) will not work * Remove the about-to-close boolean for now, keep it simple, work under a lock. It may be fine since destroy() is not a critical operation. * Create the singleton from HelixServerStarter, and start it as well. * Handle the case when segments with text index may be moved away to another host. We don't want that the thread continues to poll in this case. One way is to introduce a condition variable on the length of the queue.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
