Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dsync generated code does not work with MySQL #46

Open
longsleep opened this issue May 22, 2023 · 0 comments
Open

Dsync generated code does not work with MySQL #46

longsleep opened this issue May 22, 2023 · 0 comments

Comments

@longsleep
Copy link
Collaborator

The generated code from dsync (for create and update) uses get_result which implicitly adds a RETURNING to the generated SQL statement. MySQL does not support RETURNING hence, the trait check for generated code fails as the MysqlConnection does not implement the SupportsReturningClause marker trait.

Instead the create and update functions should use execute and leave it to the caller if it is needed to actually load the data after insert or update.

longsleep added a commit to longsleep-io/dsync that referenced this issue May 22, 2023
In certain cases it might be not desireable to write CRUD functions by
hand. To avoid conflict this change adds the option to skip generating
those functions. This for example is a workaround for MySQL which
currently is not supported by the generated functions or to support
unsigned integers in id fields.

Related: Wulf#46
Related: Wulf#47
longsleep added a commit to longsleep-io/dsync that referenced this issue May 22, 2023
In certain cases it might be not desireable to write CRUD functions by
hand. To avoid conflict this change adds the option to skip generating
those functions. This for example is a workaround for MySQL which
currently is not supported by the generated functions or to support
unsigned integers in id fields.

Related: Wulf#46
Related: Wulf#47
longsleep added a commit to longsleep-io/dsync that referenced this issue May 22, 2023
In certain cases it might be not desireable to write CRUD functions by
hand. To avoid conflict this change adds the option to skip generating
those functions. This for example is a workaround for MySQL which
currently is not supported by the generated functions or to support
unsigned integers in id fields.

Related: Wulf#46
Related: Wulf#47
longsleep added a commit to longsleep-io/dsync that referenced this issue May 22, 2023
In certain cases it might be not desireable to write CRUD functions by
hand. To avoid conflict this change adds the option to skip generating
those functions. This for example is a workaround for MySQL which
currently is not supported by the generated functions or to support
unsigned integers in id fields.

Related: Wulf#46
Related: Wulf#47
longsleep added a commit to longsleep-io/dsync that referenced this issue May 22, 2023
In certain cases it might be not desireable to write CRUD functions by
hand. To avoid conflict this change adds the option to skip generating
those functions. This for example is a workaround for MySQL which
currently is not supported by the generated functions or to support
unsigned integers in id fields.

Related: Wulf#46
Related: Wulf#47
longsleep added a commit to longsleep-io/dsync that referenced this issue May 22, 2023
In certain cases it might be not desireable to write CRUD functions by
hand. To avoid conflict this change adds the option to skip generating
those functions. This for example is a workaround for MySQL which
currently is not supported by the generated functions or to support
unsigned integers in id fields.

Related: Wulf#46
Related: Wulf#47
longsleep added a commit to longsleep-io/dsync that referenced this issue May 22, 2023
In certain cases it might be not desireable to write CRUD functions by
hand. To avoid conflict this change adds the option to skip generating
those functions. This for example is a workaround for MySQL which
currently is not supported by the generated functions or to support
unsigned integers in id fields.

Related: Wulf#46
Related: Wulf#47
longsleep added a commit to longsleep-io/dsync that referenced this issue Aug 31, 2023
In certain cases it might be not desireable to write CRUD functions by
hand. To avoid conflict this change adds the option to skip generating
those functions. This for example is a workaround for MySQL which
currently is not supported by the generated functions or to support
unsigned integers in id fields.

Related: Wulf#46
Related: Wulf#47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant