Skip to content

Conversation

@Murali1459
Copy link

Add support for query_parameter_limit configuration option

This PR implements support for the query_parameter_limit configuration option, which enables automatic generation of a data model as method input when the number of query parameters exceeds the specified limit.

@tandemdude
Copy link
Owner

Thanks for your PR! I'll give it a test once I'm home from work.

Copy link
Contributor

@davfsa davfsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey!

Thanks for your PR and sorry it took so long for it to be reviewed. I gave it a try and found that its not actually functional. It will generate the model correctly, but it doesn't properly build the statement.

Image

Note the lack of stmt.setXXX calls

@Murali1459
Copy link
Author

@tandemdude I’ve implemented the requested changes. Could you please review them?

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

Successfully merging this pull request may close these issues.

3 participants