[GitHub] [pinot] walterddr commented on a diff in pull request #8884: specify how many segments were pruned by each server segment pruner

[GitBox <gi...@apache.org>]

walterddr commented on code in PR #8884:
URL: https://github.com/apache/pinot/pull/8884#discussion_r898106187


##########
pinot-controller/src/main/resources/app/pages/Query.tsx:
##########
@@ -295,10 +297,22 @@ const QueryPage = () => {
     setResultData(results.result || { columns: [], records: [] });
     setQueryStats(results.queryStats || { columns: responseStatCols, records: [] });
     setOutputResult(JSON.stringify(results.data, null, 2) || '');
+    setWarnings(extractWarnings(results));
     setQueryLoader(false);
     queryExecuted.current = false;
   };
 
+  const extractWarnings = (result) => {
+    const warnings: Array<string> = [];
+    const numSegmentsPrunedInvalid = result.data.numSegmentsPrunedInvalid;
+    if (numSegmentsPrunedInvalid) {
+      warnings.push(`There are ${numSegmentsPrunedInvalid} invalid segment/s. This usually means that they were `
+         + `created with an older schema. `
+         + `Data wasn't lost, but these segments will be ignored by some queries until the table is reloaded.`);

Review Comment:
   ```suggestion
            + `Please reload the table in order to refresh these segments to the new schema.`);
   ```



-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org