Do not rewrite SCHEME_ERROR ststus from scheme executor #2833
Conversation
|
⚪
|
|
⚪
|
| ui64 ResultRowsBytes = 0; | ||
|
|
||
| explicit TEvTxResponse(TTxAllocatorState::TPtr allocState) | ||
| enum class EExecutionType { |
There was a problem hiding this comment.
А enum из протобуфа ты решил не использовать?
There was a problem hiding this comment.
А там кажется нет прям идеально подходящего по семантике.
в TKqpPhyQuery нет Scheme и он больше про запрос (есть отдельно SCRIPT)
TKqpPhyTx в принципе можно, но TKqpExecuterBase уже принимал enum ExecType вторым аргументом который прям хорошо ложился.
Наверно можно порефакторить чтоб использовать везде один enum (в том числе и в TKqpExecuterBase) но я что то не уверен сходу...
3 participants
Changelog category
Additional information
...