itschrispeck commented on code in PR #13094:
URL: https://github.com/apache/pinot/pull/13094#discussion_r1591820799
##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/realtime/impl/invertedindex/RealtimeLuceneTextIndex.java:
##########
@@ -151,9 +151,9 @@ public MutableRoaringBitmap getDocIds(String searchQuery) {
return searchFuture.get();
} catch (InterruptedException e) {
docIDCollector.markShouldCancel();
- LOGGER.warn("TEXT_MATCH query timeout on realtime consuming segment {},
column {}, search query {}", _segmentName,
- _column, searchQuery);
- throw new RuntimeException("TEXT_MATCH query timeout on realtime
consuming segment");
+ LOGGER.warn("TEXT_MATCH query interrupted while querying the consuming
segment {}, column {}, search query {}",
Review Comment:
I changed this message to `timeout` due to some concerns in the original PR,
but it's also used during early termination - and calling those queries
timeouts is misleading.
--
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.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
