Skip to content

Conversation

@Arp-G
Copy link
Owner

@Arp-G Arp-G commented Oct 28, 2022

We must dynamically infer the column's varchar size by finding the max data size in that column.
So the varchar_limit the user specifies must only be used to decide when to switch from varchar to text.

Helps in fixing: #25

Also contains some minor cleanup and fixes.

@Arp-G
Copy link
Owner Author

Arp-G commented Oct 29, 2022

for empty columns set varchar(1)

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.

2 participants