Review fixes

- fix style
- add descriptive comment about the nature of the storage pool kind mapping

Author: jepett0

ydb/core/tx/schemeshard/schemeshard__operation_common_subdomain.h

@@ -6,10 +6,11 @@
 namespace NKikimr {
 namespace NSchemeShard {
 
-inline bool CheckStoragePoolsInQuotas(const Ydb::Cms::DatabaseQuotas& quotas,
-                                      const TVector<TStoragePool>& pools,
-                                      const TString& path,
-                                      TString& error
+inline bool CheckStoragePoolsInQuotas(
+    const Ydb::Cms::DatabaseQuotas& quotas,
+    const TVector<TStoragePool>& pools,
+    const TString& path,
+    TString& error
 ) {
     TVector<TString> quotedKinds;
     quotedKinds.reserve(quotas.storage_quotas_size());

ydb/core/tx/schemeshard/schemeshard_info_types.cpp

@@ -51,6 +51,12 @@ EDiskUsageStatus CheckStoragePoolsQuotas(const THashMap<TString, TStoragePoolUsa
             : EDiskUsageStatus::BelowSoftQuota;
 }
 
+/*
+This is a workaround!
+It makes sense only in the specific case where:
+    - there are no multiple storage pools of the same kind
+    - all storage pool kinds belong to the set {ssh*, hdd*, rot*, nvme*}
+*/
 EUserFacingStorageType GetUserFacingStorageType(const TString& poolKind) {
     if (poolKind.StartsWith("ssd") || poolKind.StartsWith("nvme")) {
         return EUserFacingStorageType::Ssd;