Benjamin Schultz Larsen
Benjamin Schultz Larsen
That decision would I leave completely up to you, it sound like that its going to change for the next version regardless to bridge the compatible gap
if you have been inspired by the Postgres logic, make sure you haven't been falling into this trap #3837
Linked to issue #3837 I can see there is a lot of tests there fails, the question should then be if the new output is the desired or not? Is...
@theAnuragMishra maybe you could update the tests so it's easier to see the impact of this change? It needs to be updated anyway for it to get merged
@acartine I think you'd be more than welcome to give it a go in a new PR
@acartine Issnt that what "emit_pointers_for_null_types" is suppose to configure? [Docs](https://docs.sqlc.dev/en/latest/reference/config.html#:~:text=If%20true%2C%20generated%20types%20for%20nullable%20columns%20are%20emitted%20as%20pointers) > - emit_pointers_for_null_types: > - If true, generated types for nullable columns are emitted as pointers (ie. *string) instead of...
@acartine lets move the discussion to the issue but to sum up short - i dont know the mindset balancing breaking changes and configs, we need some attention from Kyle...
It's great to see iterators coming to sqlc. Adding memory pooling as an option on top of this would make this truly a game-changer for large result sets. I would...