search

shuttle-hq / synth

The Declarative Data Generator
https://www.getsynth.com/
Apache License 2.0
1.37k stars 108 forks source link

feat: start work on uuid support #411

Open iamwacko opened 1 year ago

iamwacko commented 1 year ago

UUID was implemented as a string, which meant it would not work with the UUID type in databases. #270

iamwacko commented 1 year ago

If anyone wants this merged they have to figure out how to make pretty printing the OwnedSerializable in the test_compile_and_generate test in the core crate not give an error along the lines of key must be a string.

ratskates commented 1 year ago

Any chance of getting this merged in ? It's a read blocker for me.

JRileyH commented 1 year ago

What's the realistic timeline of getting this merged? I cannot use synth until UUIDs are supported in postgres.

Also thanks for taking his on @iamwacko :) Looks like more than a few people will be really happy to see this fixed.

iamwacko commented 1 year ago

I don't really have a schedule. It should be working, and rust-gdb even shows UUID's in memory. The only blocker is the key must be a string error, which I have spent hours trying to fix. If anyone could figure out how to deal with it I could put out a new release that day.

njohnson-ml commented 1 year ago

@iamwacko do you get any more info on the failure when you set RUST_BACKTRACE=1?

murugesanp92 commented 8 months ago

@iamwacko any progress on this?