Skip to content

Commit

Permalink
basic statistics bugfixes KIKIMR-18323 (ydb-platform#915)
Browse files Browse the repository at this point in the history
  • Loading branch information
alexd65536 authored Jan 12, 2024
1 parent 2c3d1e3 commit 8206290
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
2 changes: 2 additions & 0 deletions ydb/core/cms/console/console__create_tenant.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -217,6 +217,8 @@ class TTenantsManager::TTxCreateTenant : public TTransactionBase<TTenantsManager
break;

case Ydb::Cms::CreateDatabaseRequest::kServerlessResources:
Tenant->IsExternalStatisticsAggregator = false;

if (!Tenant->IsExternalSubdomain) {
return Error(Ydb::StatusIds::PRECONDITION_FAILED,
"Cannot create serverless database unless external subdomain is enabled", ctx);
Expand Down
4 changes: 4 additions & 0 deletions ydb/core/statistics/aggregator/aggregator_impl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,10 @@ void TStatisticsAggregator::Handle(TEvStatistics::TEvConnectNode::TPtr& ev) {
IsPropagateInFlight = true;
}

if (!record.NeedSchemeShardsSize()) {
return;
}

std::vector<TSSId> ssIds;
ssIds.reserve(record.NeedSchemeShardsSize());
for (const auto& ssId : record.GetNeedSchemeShards()) {
Expand Down

0 comments on commit 8206290

Please sign in to comment.