Skip to content

Commit 36cecdb

Browse files
nikvas0Kamil Khamitov
authored andcommitted
Fix EvWrite Issues data (ydb-platform#15156)
1 parent 83821f3 commit 36cecdb

File tree

2 files changed

+12
-6
lines changed

2 files changed

+12
-6
lines changed

ydb/core/kqp/ut/yql/kqp_yql_ut.cpp

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -202,8 +202,10 @@ Y_UNIT_TEST_SUITE(KqpYql) {
202202
UNIT_ASSERT(HasIssue(result.GetIssues(), NYql::TIssuesIds::DEFAULT_ERROR));
203203
}
204204

205-
Y_UNIT_TEST(InsertCV) {
206-
auto kikimr = DefaultKikimrRunner();
205+
Y_UNIT_TEST_TWIN(InsertCV, useSink) {
206+
NKikimrConfig::TAppConfig appConfig;
207+
appConfig.MutableTableServiceConfig()->SetEnableOltpSink(useSink);
208+
auto kikimr = DefaultKikimrRunner({}, appConfig);
207209
TScriptingClient client(kikimr.GetDriver());
208210

209211
auto result = client.ExecuteYqlScript(R"(
@@ -217,8 +219,10 @@ Y_UNIT_TEST_SUITE(KqpYql) {
217219
UNIT_ASSERT(HasIssue(result.GetIssues(), NYql::TIssuesIds::KIKIMR_CONSTRAINT_VIOLATION));
218220
}
219221

220-
Y_UNIT_TEST(InsertCVList) {
221-
auto kikimr = DefaultKikimrRunner();
222+
Y_UNIT_TEST_TWIN(InsertCVList, useSink) {
223+
NKikimrConfig::TAppConfig appConfig;
224+
appConfig.MutableTableServiceConfig()->SetEnableOltpSink(useSink);
225+
auto kikimr = DefaultKikimrRunner({}, appConfig);
222226
TScriptingClient client(kikimr.GetDriver());
223227

224228
auto result = client.ExecuteYqlScript(R"(

ydb/core/tx/data_events/events.h

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@
1111
#include <ydb/library/actors/core/event_pb.h>
1212
#include <ydb/library/actors/core/log.h>
1313

14+
#include <yql/essentials/public/issue/yql_issue_message.h>
15+
1416
namespace NKikimr::NEvents {
1517

1618
struct TDataEvents {
@@ -100,8 +102,8 @@ struct TDataEvents {
100102
result->Record.SetOrigin(origin);
101103
result->Record.SetTxId(txId);
102104
result->Record.SetStatus(status);
103-
auto issue = result->Record.AddIssues();
104-
issue->set_message(errorMsg);
105+
NYql::TIssue issue(errorMsg);
106+
NYql::IssueToMessage(issue, result->Record.AddIssues());
105107
return result;
106108
}
107109

0 commit comments

Comments
 (0)