diff options
author | hcpp <hcpp@yandex-team.ru> | 2022-03-23 19:57:58 +0300 |
---|---|---|
committer | hcpp <hcpp@yandex-team.ru> | 2022-03-23 19:57:58 +0300 |
commit | 97b04c23a5839ba7768ceae283cfd6fcee190ac5 (patch) | |
tree | e1fe99fffdee4989a500fa8ceb5de05a5a848bdd | |
parent | 6edc01c005ae7d58c9c1ceaa0c0e260918514522 (diff) | |
download | ydb-97b04c23a5839ba7768ceae283cfd6fcee190ac5.tar.gz |
clang tidy for ydb yq
ref:d8e3f0fe5547b878b9f6ed7f0124c8ad192311d1
-rw-r--r-- | ydb/core/yq/.clang-tidy | 25 | ||||
-rw-r--r-- | ydb/core/yq/libs/actors/pending_fetcher.cpp | 6 |
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 }; } } |