diff options
| author | robot-piglet <robot-piglet@yandex-team.com> | 2024-02-19 19:03:57 +0300 |
|---|---|---|
| committer | robot-piglet <robot-piglet@yandex-team.com> | 2024-02-19 19:22:10 +0300 |
| commit | 119e8024f38bae4f04fd372110f45ddb2c01d71f (patch) | |
| tree | 4bad5cc40a4c9090ba4005561745ca559d00da6f | |
| parent | 793447a2b4ab1e8ac2b8d1773c8cbbc0e34d31ab (diff) | |
| download | ydb-119e8024f38bae4f04fd372110f45ddb2c01d71f.tar.gz | |
Intermediate changes
| -rw-r--r-- | contrib/libs/llvm16/lib/IR/LegacyPassManager.cpp | 5 | ||||
| -rw-r--r-- | contrib/python/types-protobuf/ya.make | 12 | ||||
| -rw-r--r-- | yt/yt/client/api/rpc_proxy/helpers.cpp | 6 |
3 files changed, 8 insertions, 15 deletions
diff --git a/contrib/libs/llvm16/lib/IR/LegacyPassManager.cpp b/contrib/libs/llvm16/lib/IR/LegacyPassManager.cpp index ef34651776..f9572cc246 100644 --- a/contrib/libs/llvm16/lib/IR/LegacyPassManager.cpp +++ b/contrib/libs/llvm16/lib/IR/LegacyPassManager.cpp @@ -1414,7 +1414,10 @@ bool FPPassManager::runOnFunction(Function &F) { FunctionPass *FP = getContainedPass(Index); bool LocalChanged = false; - llvm::TimeTraceScope PassScope("RunPass", FP->getPassName()); + // Call getPassName only when required. The call itself is fairly cheap, but + // still virtual and repeated calling adds unnecessary overhead. + llvm::TimeTraceScope PassScope( + "RunPass", [FP]() { return std::string(FP->getPassName()); }); dumpPassInfo(FP, EXECUTION_MSG, ON_FUNCTION_MSG, F.getName()); dumpRequiredSet(FP); diff --git a/contrib/python/types-protobuf/ya.make b/contrib/python/types-protobuf/ya.make index 6632bf3426..9ba2128ebd 100644 --- a/contrib/python/types-protobuf/ya.make +++ b/contrib/python/types-protobuf/ya.make @@ -11,16 +11,9 @@ NO_LINT() PY_SRCS( TOP_LEVEL google-stubs/protobuf/__init__.pyi - google-stubs/protobuf/any_pb2.pyi - google-stubs/protobuf/api_pb2.pyi google-stubs/protobuf/compiler/__init__.pyi - google-stubs/protobuf/compiler/plugin_pb2.pyi google-stubs/protobuf/descriptor.pyi - google-stubs/protobuf/descriptor_pb2.pyi google-stubs/protobuf/descriptor_pool.pyi - google-stubs/protobuf/duration_pb2.pyi - google-stubs/protobuf/empty_pb2.pyi - google-stubs/protobuf/field_mask_pb2.pyi google-stubs/protobuf/internal/__init__.pyi google-stubs/protobuf/internal/api_implementation.pyi google-stubs/protobuf/internal/builder.pyi @@ -39,14 +32,9 @@ PY_SRCS( google-stubs/protobuf/message_factory.pyi google-stubs/protobuf/reflection.pyi google-stubs/protobuf/service.pyi - google-stubs/protobuf/source_context_pb2.pyi - google-stubs/protobuf/struct_pb2.pyi google-stubs/protobuf/symbol_database.pyi google-stubs/protobuf/text_format.pyi - google-stubs/protobuf/timestamp_pb2.pyi - google-stubs/protobuf/type_pb2.pyi google-stubs/protobuf/util/__init__.pyi - google-stubs/protobuf/wrappers_pb2.pyi ) RESOURCE_FILES( diff --git a/yt/yt/client/api/rpc_proxy/helpers.cpp b/yt/yt/client/api/rpc_proxy/helpers.cpp index 45a5c48d6a..677e8b9bc0 100644 --- a/yt/yt/client/api/rpc_proxy/helpers.cpp +++ b/yt/yt/client/api/rpc_proxy/helpers.cpp @@ -1662,12 +1662,14 @@ NJobTrackerClient::EJobState ConvertJobStateFromProto( bool IsDynamicTableRetriableError(const TError& error) { - // TODO(dgolear): Consider adding NoSuchTablet and TabletNotMounted errors? return error.FindMatching(NTabletClient::EErrorCode::RowIsBlocked) || error.FindMatching(NTabletClient::EErrorCode::BlockedRowWaitTimeout) || error.FindMatching(NTabletClient::EErrorCode::NoSuchCell) || - error.FindMatching(NTabletClient::EErrorCode::ChunkIsNotPreloaded); + error.FindMatching(NTabletClient::EErrorCode::ChunkIsNotPreloaded) || + error.FindMatching(NTabletClient::EErrorCode::NoInSyncReplicas) || + error.FindMatching(NTabletClient::EErrorCode::TabletNotMounted) || + error.FindMatching(NTabletClient::EErrorCode::NoSuchTablet); } bool IsRetriableError(const TError& error, bool retryProxyBanned) |