summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorudovichenko-r <[email protected]>2022-08-15 14:44:27 +0300
committerudovichenko-r <[email protected]>2022-08-15 14:44:27 +0300
commit6ad0163d8a1e258b75889ed4e1143ab1301fc6b0 (patch)
tree65064f47ef0b18e6ad70ebab0e563f02a14408c0
parentfbe5e43134fdc57152afea6b9bf1830a733a1f3a (diff)
[yql] Add info about DQ fallback
-rw-r--r--ydb/library/yql/providers/dq/provider/yql_dq_recapture.cpp12
1 files changed, 7 insertions, 5 deletions
diff --git a/ydb/library/yql/providers/dq/provider/yql_dq_recapture.cpp b/ydb/library/yql/providers/dq/provider/yql_dq_recapture.cpp
index cf7c7aaf2d5..25d7c45d76b 100644
--- a/ydb/library/yql/providers/dq/provider/yql_dq_recapture.cpp
+++ b/ydb/library/yql/providers/dq/provider/yql_dq_recapture.cpp
@@ -80,17 +80,19 @@ public:
TNodeSet visited;
Scan(*input, ctx, good, dataSize, visited, hasJoin);
+ if (good && hasJoin && dataSize > State_->Settings->MaxDataSizePerQuery.Get().GetOrElse(10_GB)) {
+ Statistics_["DqAnalyzerBigJoin"]++;
+ AddInfo(ctx, TStringBuilder() << "too big join input: " << dataSize);
+ good = false;
+ }
+
if (good) {
Statistics_["DqAnalyzerOk"]++;
} else {
Statistics_["DqAnalyzerFail"] ++;
}
- if ((hasJoin && dataSize > State_->Settings->MaxDataSizePerQuery.Get().GetOrElse(10_GB))) {
- Statistics_["DqAnalyzerBigJoin"]++;
- }
-
- if (!good || (hasJoin && dataSize > State_->Settings->MaxDataSizePerQuery.Get().GetOrElse(10_GB))) {
+ if (!good) {
YQL_CLOG(DEBUG, ProviderDq) << "good: " << good << " hasJoin: " << hasJoin << " dataSize: " << dataSize;
return TStatus::Ok;
}