aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorhcpp <hcpp@yandex-team.ru>2022-03-23 19:57:58 +0300
committerhcpp <hcpp@yandex-team.ru>2022-03-23 19:57:58 +0300
commit97b04c23a5839ba7768ceae283cfd6fcee190ac5 (patch)
treee1fe99fffdee4989a500fa8ceb5de05a5a848bdd
parent6edc01c005ae7d58c9c1ceaa0c0e260918514522 (diff)
downloadydb-97b04c23a5839ba7768ceae283cfd6fcee190ac5.tar.gz
clang tidy for ydb yq
ref:d8e3f0fe5547b878b9f6ed7f0124c8ad192311d1
-rw-r--r--ydb/core/yq/.clang-tidy25
-rw-r--r--ydb/core/yq/libs/actors/pending_fetcher.cpp6
2 files changed, 28 insertions, 3 deletions
diff --git a/ydb/core/yq/.clang-tidy b/ydb/core/yq/.clang-tidy
new file mode 100644
index 00000000000..b96170b0fb2
--- /dev/null
+++ b/ydb/core/yq/.clang-tidy
@@ -0,0 +1,25 @@
+Checks: >
+ -*,
+ arcadia-typeid-name-restriction,
+ bugprone-use-after-move,
+ readability-identifier-naming,
+CheckOptions:
+ - key: readability-identifier-naming.ClassCase
+ value: CamelCase
+ - key: readability-identifier-naming.PublicMemberCase
+ value: CamelCase
+ - key: readability-identifier-naming.ProtectedMemberCase
+ value: CamelCase
+ - key: readability-identifier-naming.PrivateMemberCase
+ value: CamelCase
+ - key: readability-identifier-naming.FunctionCase
+ value: CamelCase
+ # do not tide public begin / end methods to workaround clang-tidy bug in range-based for loops
+ - key: readability-identifier-naming.PublicMethodIgnoredRegexp
+ value: "(begin|end|empty|size|ysize|front|back)"
+ - key: readability-identifier-naming.PublicMethodCase
+ value: CamelCase
+ - key: readability-identifier-naming.ProtectedMethodCase
+ value: CamelCase
+ - key: readability-identifier-naming.PrivateMethodCase
+ value: CamelCase
diff --git a/ydb/core/yq/libs/actors/pending_fetcher.cpp b/ydb/core/yq/libs/actors/pending_fetcher.cpp
index f24225aaf28..95dda9c762a 100644
--- a/ydb/core/yq/libs/actors/pending_fetcher.cpp
+++ b/ydb/core/yq/libs/actors/pending_fetcher.cpp
@@ -369,9 +369,9 @@ private:
auto runActorId = Register(CreateRunActor(SelfId(), queryCounters, std::move(params)));
- RunActorMap[runActorId] = params.QueryId;
- if (!params.Automatic) {
- CountersMap[params.QueryId] = { rootCountersParent, publicCountersParent, runActorId };
+ RunActorMap[runActorId] = queryId;
+ if (!task.automatic()) {
+ CountersMap[queryId] = { rootCountersParent, publicCountersParent, runActorId };
}
}