Skip to content

Commit

Permalink
Merge 6091636 into 930e312
Browse files Browse the repository at this point in the history
  • Loading branch information
swalrus1 authored Oct 23, 2024
2 parents 930e312 + 6091636 commit d9612a7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ydb/core/ydb_convert/ydb_convert.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -513,7 +513,7 @@ Y_FORCE_INLINE void ConvertData(NUdf::TDataTypeId typeId, const Ydb::Value& valu
CheckTypeId(value.value_case(), Ydb::Value::kTextValue, "JsonDocument");
const auto binaryJson = NBinaryJson::SerializeToBinaryJson(value.text_value());
if (binaryJson.IsFail()) {
throw yexception() << "Invalid JsonDocument value";
throw yexception() << "Invalid JsonDocument value: " << binaryJson.GetErrorMessage();
}
res.SetBytes(binaryJson->Data(), binaryJson->Size());
break;
Expand Down Expand Up @@ -1239,7 +1239,7 @@ bool CellFromProtoVal(const NScheme::TTypeInfo& type, i32 typmod, const Ydb::Val
case NScheme::NTypeIds::JsonDocument : {
const auto binaryJson = NBinaryJson::SerializeToBinaryJson(val.Gettext_value());
if (binaryJson.IsFail()) {
err = "Invalid JSON for JsonDocument provided";
err = "Invalid JSON for JsonDocument provided: " + binaryJson.GetErrorMessage();
return false;
}
const auto binaryJsonInPool = valueDataPool.AppendString(TStringBuf(binaryJson->Data(), binaryJson->Size()));
Expand Down

0 comments on commit d9612a7

Please sign in to comment.