Re: [PR] correct errmsg format for Preconditions.check... [pinot]

2024-01-26 Thread via GitHub
klsince merged PR #12327: URL: https://github.com/apache/pinot/pull/12327 -- 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.apac

[PR] correct errmsg format for Preconditions.check... [pinot]

2024-01-26 Thread via GitHub
klsince opened a new pull request, #12327: URL: https://github.com/apache/pinot/pull/12327 Preconditions.check... errmsg uses `%s` to format instead of {}, otherwise, we'd see errmsg like below ``` java.lang.IllegalStateException: Snapshot of validDocIds is required to preload seg