You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I was surprised to see UUID columns represented as string in generated Go. The CockroachDB Manual clearly states that this data is stored as bytes, and in my Go code, I pass them around as []byte as string conversion is a completely unnecessary waste of resources.
I realize this would be a breaking change, but can we make it configurable? Thanks for looking into this.
The text was updated successfully, but these errors were encountered:
@willbuckner I don't have time right now but I found it was already discussed here: volatiletech/sqlboiler#58, maybe ask in the core library first and depending on the status and reasoning we'll proceed with this or not.
sqlboiler-crdb/driver/crdb.go
Lines 348 to 349 in d74f22d
I was surprised to see
UUID
columns represented asstring
in generated Go. The CockroachDB Manual clearly states that this data is stored as bytes, and in my Go code, I pass them around as[]byte
as string conversion is a completely unnecessary waste of resources.I realize this would be a breaking change, but can we make it configurable? Thanks for looking into this.
The text was updated successfully, but these errors were encountered: