diff --git a/src/carnot/exec/grpc_router.h b/src/carnot/exec/grpc_router.h index 37303a4676b..ba40c66dd97 100644 --- a/src/carnot/exec/grpc_router.h +++ b/src/carnot/exec/grpc_router.h @@ -110,14 +110,14 @@ class GRPCRouter final : public carnotpb::ResultSinkService::Service { */ struct SourceNodeTracker { SourceNodeTracker() = default; - GRPCSourceNode* source_node GUARDED_BY(node_lock) = nullptr; + GRPCSourceNode* source_node ABSL_GUARDED_BY(node_lock) = nullptr; // connection_initiated_by_sink and connection_closed_by_sink are true when the // grpc sink (aka the client) initiates the query result stream or closes a query result stream, // respectively. - bool connection_initiated_by_sink GUARDED_BY(node_lock) = false; - bool connection_closed_by_sink GUARDED_BY(node_lock) = false; + bool connection_initiated_by_sink ABSL_GUARDED_BY(node_lock) = false; + bool connection_closed_by_sink ABSL_GUARDED_BY(node_lock) = false; std::vector> response_backlog - GUARDED_BY(node_lock); + ABSL_GUARDED_BY(node_lock); absl::base_internal::SpinLock node_lock; }; @@ -126,17 +126,18 @@ class GRPCRouter final : public carnotpb::ResultSinkService::Service { */ struct QueryTracker { QueryTracker() : create_time(std::chrono::steady_clock::now()) {} - absl::node_hash_map source_node_trackers GUARDED_BY(query_lock); - const std::chrono::steady_clock::time_point create_time GUARDED_BY(query_lock); - std::function restart_execution_func_ GUARDED_BY(query_lock); + absl::node_hash_map source_node_trackers + ABSL_GUARDED_BY(query_lock); + const std::chrono::steady_clock::time_point create_time ABSL_GUARDED_BY(query_lock); + std::function restart_execution_func_ ABSL_GUARDED_BY(query_lock); // The set of agents we've seen for the query. - absl::flat_hash_set seen_agents GUARDED_BY(query_lock); - absl::flat_hash_set<::grpc::ServerContext*> active_agent_contexts GUARDED_BY(query_lock); + absl::flat_hash_set seen_agents ABSL_GUARDED_BY(query_lock); + absl::flat_hash_set<::grpc::ServerContext*> active_agent_contexts ABSL_GUARDED_BY(query_lock); // The execution stats for agents that are clients to this service. - std::vector agent_exec_stats GUARDED_BY(query_lock); + std::vector agent_exec_stats ABSL_GUARDED_BY(query_lock); // Errors that occur during execution from parent_agents. - std::vector upstream_exec_errors GUARDED_BY(query_lock); + std::vector upstream_exec_errors ABSL_GUARDED_BY(query_lock); absl::base_internal::SpinLock query_lock; void ResetRestartExecutionFunc() ABSL_EXCLUSIVE_LOCKS_REQUIRED(query_lock) { @@ -178,7 +179,7 @@ class GRPCRouter final : public carnotpb::ResultSinkService::Service { SourceNodeTracker* GetSourceNodeTracker(QueryTracker* query_tracker, int64_t source_id); absl::node_hash_map> id_to_query_tracker_map_ - GUARDED_BY(id_to_query_tracker_map_lock_); + ABSL_GUARDED_BY(id_to_query_tracker_map_lock_); mutable absl::base_internal::SpinLock id_to_query_tracker_map_lock_; }; diff --git a/src/carnot/udf/borrow_pool.h b/src/carnot/udf/borrow_pool.h index ef96a0b1a2d..92e3b8e82a0 100644 --- a/src/carnot/udf/borrow_pool.h +++ b/src/carnot/udf/borrow_pool.h @@ -66,7 +66,7 @@ class BorrowPool { } private: - std::vector pool_ GUARDED_BY(pool_lock_); + std::vector pool_ ABSL_GUARDED_BY(pool_lock_); absl::base_internal::SpinLock pool_lock_; }; diff --git a/src/experimental/standalone_pem/sink_server.h b/src/experimental/standalone_pem/sink_server.h index 4f38e69e9b7..eb822009fe8 100644 --- a/src/experimental/standalone_pem/sink_server.h +++ b/src/experimental/standalone_pem/sink_server.h @@ -180,7 +180,7 @@ class StandaloneResultSinkServer final : public carnotpb::ResultSinkService::Ser } absl::flat_hash_map*> - consumer_map_ GUARDED_BY(id_to_query_consumer_map_lock_); + consumer_map_ ABSL_GUARDED_BY(id_to_query_consumer_map_lock_); mutable absl::base_internal::SpinLock id_to_query_consumer_map_lock_; }; diff --git a/src/vizier/services/agent/shared/manager/chan_cache.h b/src/vizier/services/agent/shared/manager/chan_cache.h index 00106a2e2f4..6520a1c03a9 100644 --- a/src/vizier/services/agent/shared/manager/chan_cache.h +++ b/src/vizier/services/agent/shared/manager/chan_cache.h @@ -83,7 +83,7 @@ class ChanCache { }; // The cache of channels (grpc conns) made to other agents. - absl::flat_hash_map chan_cache_ GUARDED_BY(chan_cache_lock_); + absl::flat_hash_map chan_cache_ ABSL_GUARDED_BY(chan_cache_lock_); absl::base_internal::SpinLock chan_cache_lock_; // Connections that are alive for shorter than warm_up_period_ won't be cleared. std::chrono::nanoseconds warm_up_period_; diff --git a/src/vizier/services/agent/shared/manager/relation_info_manager.h b/src/vizier/services/agent/shared/manager/relation_info_manager.h index 10c05039328..f4cf1080e3d 100644 --- a/src/vizier/services/agent/shared/manager/relation_info_manager.h +++ b/src/vizier/services/agent/shared/manager/relation_info_manager.h @@ -65,7 +65,8 @@ class RelationInfoManager { private: mutable std::atomic has_updates_ = false; mutable absl::base_internal::SpinLock relation_info_map_lock_; - absl::btree_map relation_info_map_ GUARDED_BY(relation_info_map_lock_); + absl::btree_map relation_info_map_ + ABSL_GUARDED_BY(relation_info_map_lock_); }; } // namespace agent