Skip to content

Commit 8949c45

Browse files
committed
Fixed unit test
1 parent f9f3237 commit 8949c45

File tree

1 file changed

+22
-3
lines changed

1 file changed

+22
-3
lines changed

ydb/core/kqp/ut/federated_query/s3/kqp_federated_query_ut.cpp

+22-3
Original file line numberDiff line numberDiff line change
@@ -1756,10 +1756,29 @@ Y_UNIT_TEST_SUITE(KqpFederatedQuery) {
17561756
auto status = operationClient.Forget(scriptExecutionOperation.Id()).ExtractValueSync();
17571757
UNIT_ASSERT_C(status.IsSuccess(), status.GetIssues().ToOneLineString());
17581758

1759-
const size_t forgetRowsLimit = 100000;
1760-
if (numberRows > forgetRowsLimit) {
1761-
UNIT_ASSERT_STRING_CONTAINS(status.GetIssues().ToString(), TStringBuilder() << "Info: Query result rows count is " << numberRows << ", that is larger than allowed limit " << forgetRowsLimit << " rows for one time forget, results will be forgotten in the background process");
1759+
const TString countResultsQuery = fmt::format(R"(
1760+
SELECT COUNT(*)
1761+
FROM `.metadata/result_sets`
1762+
WHERE execution_id = "{execution_id}" AND expire_at > CurrentUtcTimestamp();
1763+
)", "execution_id"_a=readyOp.Metadata().ExecutionId);
1764+
1765+
TInstant forgetChecksStart = TInstant::Now();
1766+
while (TInstant::Now() - forgetChecksStart <= TDuration::Minutes(5)) {
1767+
NYdb::NTable::TDataQueryResult result = session.ExecuteDataQuery(countResultsQuery, NYdb::NTable::TTxControl::BeginTx().CommitTx()).ExtractValueSync();
1768+
UNIT_ASSERT_C(result.IsSuccess(), result.GetIssues().ToString());
1769+
1770+
auto resultSet = result.GetResultSetParser(0);
1771+
resultSet.TryNextRow();
1772+
1773+
ui64 numberRows = resultSet.ColumnParser(0).GetUint64();
1774+
if (!numberRows) {
1775+
return;
1776+
}
1777+
1778+
Cerr << "Rows remains: " << numberRows << ", elapsed time: " << TInstant::Now() - forgetChecksStart << "\n";
1779+
Sleep(TDuration::Seconds(1));
17621780
}
1781+
UNIT_ASSERT_C(false, "Results removing timeout");
17631782
}
17641783

17651784
Y_UNIT_TEST(ExecuteScriptWithLargeStrings) {

0 commit comments

Comments
 (0)