diff options
| author | ijon <ijon@yandex-team.com> | 2022-12-28 16:37:02 +0300 |
|---|---|---|
| committer | ijon <ijon@yandex-team.com> | 2022-12-28 16:37:02 +0300 |
| commit | 44cea36bf7524702d2849df9a4f52933f5e7d355 (patch) | |
| tree | a23071927778ac950c0c3aff407b27158323fc9f | |
| parent | 7685775b0004801ef7e304eb195881a4831f4ec5 (diff) | |
| download | ydb-44cea36bf7524702d2849df9a4f52933f5e7d355.tar.gz | |
schemeshard: revert spelling fix for "path version mistmach"
There is dependency on that exact text in the nbs code.
| -rw-r--r-- | ydb/core/tx/schemeshard/schemeshard_impl.cpp | 4 | ||||
| -rw-r--r-- | ydb/core/tx/schemeshard/ut_helpers/ls_checks.cpp | 8 |
2 files changed, 9 insertions, 3 deletions
diff --git a/ydb/core/tx/schemeshard/schemeshard_impl.cpp b/ydb/core/tx/schemeshard/schemeshard_impl.cpp index 08bb94357d5..c67067c54bc 100644 --- a/ydb/core/tx/schemeshard/schemeshard_impl.cpp +++ b/ydb/core/tx/schemeshard/schemeshard_impl.cpp @@ -1129,7 +1129,9 @@ bool TSchemeShard::CheckApplyIf(const NKikimrSchemeOp::TModifyScheme &scheme, TS if (requiredVersion != actualVersion) { errStr = TStringBuilder() << "fail user constraint in ApplyIf section:" - << " path version mismatch, path with id " << pathEl->PathId + //FIXME: revert to misspelled text as there is dependency on it in the nbs code. + // Dependency on text should be replaced by introducing special error code. + << " path version mistmach, path with id " << pathEl->PathId << " has actual version " << actualVersion << " but version " << requiredVersion << " was required"; return false; diff --git a/ydb/core/tx/schemeshard/ut_helpers/ls_checks.cpp b/ydb/core/tx/schemeshard/ut_helpers/ls_checks.cpp index 657d02ee6bf..518649ac343 100644 --- a/ydb/core/tx/schemeshard/ut_helpers/ls_checks.cpp +++ b/ydb/core/tx/schemeshard/ut_helpers/ls_checks.cpp @@ -524,7 +524,9 @@ TCheckFunc PathVersionEqual(ui64 version) { const auto& curVersion = self.GetPathVersion(); UNIT_ASSERT_EQUAL_C(curVersion, version, - "path version mismatch, path with id " << self.GetPathId() << + //FIXME: revert to misspelled text as there is dependency on it in the nbs code. + // Dependency on text should be replaced by introducing special error code. + "path version mistmach, path with id " << self.GetPathId() << " has version " << curVersion << " but expected " << version); }; @@ -538,7 +540,9 @@ TCheckFunc PathVersionOneOf(TSet<ui64> versions) { const auto& curVersion = self.GetPathVersion(); UNIT_ASSERT_C(versions.count(curVersion) > 0, - "path version mismatch, path with id " << self.GetPathId() << + //FIXME: revert to misspelled text as there is dependency on it in the nbs code. + // Dependency on text should be replaced by introducing special error code. + "path version mistmach, path with id " << self.GetPathId() << " has version " << curVersion << " but expected one of set"); }; |