Skip to content

Commit

Permalink
Updating log lines from info to error when exceeded limit
Browse files Browse the repository at this point in the history
  • Loading branch information
sandeep6189 committed Nov 27, 2023
1 parent 0e49dc7 commit ee10cea
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions query/src/main/scala/filodb/query/exec/ExecPlan.scala
Original file line number Diff line number Diff line change
Expand Up @@ -289,15 +289,15 @@ trait ExecPlan extends QueryCommand {
if (enforceSampleLimit && numResultSamples > queryContext.plannerParams.enforcedLimits.execPlanSamples) {
val msg = s"Exceeded enforced limit of samples produced on a single shard or processing node. " +
s"Max number of samples is ${queryContext.plannerParams.enforcedLimits.execPlanSamples}"
qLogger.warn(queryContext.getQueryLogLine(msg))
qLogger.error(queryContext.getQueryLogLine(msg))
throw QueryLimitException(s"This query results in more than " +
s"${queryContext.plannerParams.enforcedLimits.execPlanSamples} samples. " +
s"Try applying more filters or reduce time range.", queryContext.queryId)
}
if (numResultSamples > queryContext.plannerParams.warnLimits.execPlanSamples) {
val msg = s"Exceeded warning limit of samples produced on a single shard or processing node. " +
s"Max number of samples is ${queryContext.plannerParams.warnLimits.execPlanSamples}"
qLogger.info(queryContext.getQueryLogLine(msg))
qLogger.error(queryContext.getQueryLogLine(msg))
queryWarnings.updateExecPlanSamples(numResultSamples)
}
}
Expand Down

0 comments on commit ee10cea

Please sign in to comment.