shubhamvishu commented on PR #12798: URL: https://github.com/apache/lucene/pull/12798#issuecomment-1834080104
@javanna Hmm ....I agree with you the consumers don't really depend on TE to run their Futures and its more like a generic utility function being exposed by TE(which is better to avoid now that I'm thinking more and more). Maybe this sort of function would be more suited in some utility class under `org.apache.lucene.util` but I don't see any such obvious candidate. Let me know if you think there exists some other place for this function instead of TE, otherwise we could close this PR I think. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
