all: added support for ydb in codegen#10
all: added support for ydb in codegen#10kprokopenko merged 13 commits intoydb-platform:ydb-developfrom
Conversation
LostImagin4tion
force-pushed
the
LostImagin4tion/ydb-codegen-support
branch
from
fafba72 to
621e58e
Compare
LostImagin4tion
changed the title
LostImagin4tion
force-pushed
the
LostImagin4tion/ydb-codegen-support
branch
3 times, most recently
from
b3b14ce to
a8b277a
Compare
| hooks []Hook | ||
| mutation *UserMutation | ||
| hooks []Hook | ||
| retryConfig sqlgraph.RetryConfig |
There was a problem hiding this comment.
Напишу здесь, но касается и остальных примеров.
Правильно я понял, что retryConfig и ретраи будут генерироваться для всех диалектов, а не только для YDB? Если так, то это снижает шансы на мерж в апстрим до нуля. Либо надо делать отдельным PR в апстрим.
There was a problem hiding this comment.
Правильно я понял, что retryConfig и ретраи будут генерироваться для всех диалектов
Да, все так. Но у ent-а есть механизм фичетогглов, который влияет на кодогенерацию.
Сначала я думал его использовать, но потом подумал, что это может быть странно. Потому что интерфейсы для ретраев на уровне драйвера этим фичетогглом не закрыть. То есть получается, не полностью закрываем функциональность
Мб надо подумать, как сделать получше
There was a problem hiding this comment.
Мб надо подумать, как сделать получше
ага, давай ещё раз подумаем, прежде чем мержить
There was a problem hiding this comment.
готово, закрыл кодогенерацию фичетогглом
LostImagin4tion
force-pushed
the
LostImagin4tion/ydb-codegen-support
branch
from
a8b277a to
5973afa
Compare
LostImagin4tion
force-pushed
the
LostImagin4tion/ydb-codegen-support
branch
from
5973afa to
85cf676
Compare
2 participants
No description provided.