gonymizer icon indicating copy to clipboard operation
gonymizer copied to clipboard

UUID relationship between tables not working

Open LVSant opened this issue 2 years ago • 1 comments

I have a dataset with three tables.

One main items table that uses id as uuid that are unique, and two tables with things like item_comment and item_history that references the id from the main items table.

The problem is that ALL ids from items table are 00000000-0000-0000-0000-000000000000 so I lost the reference to item_comment and item_history tables.

I've configured the relationship "parentColumn", "parentSchema" and "parentTable" on both the item table and also the item_history and item_comment tables.

How can I fix that? Thanks.

LVSant avatar Apr 21 '22 19:04 LVSant

@leonardoviveiros have a couple questions:

  1. Which version of Gonymizer are you using?
  2. Does this happen on version v1.2.3 and v2.0.0?

junkert avatar Apr 25 '22 16:04 junkert

Hey @junkert

I have the same issue using the latest version of gonymizer.. I'm trying to figure out what it the problem, let me know if you have an idea how to debug it better.

luizm avatar May 19 '23 20:05 luizm

Hi @luizm. Which version are you using? Is this on the master branch?

junkert avatar May 22 '23 14:05 junkert

Hi @luizm. Which version are you using? Is this on the master branch?

Yes.. using master

luizm avatar May 22 '23 14:05 luizm

I'll try to find some time to look into this today

junkert avatar May 22 '23 14:05 junkert

So sorry this has been broken, I'm surprised that testing was working during this failure :( I'll look into this as well.

I was finally able to get a dev setup working again. I've been able to pin-point the issue to somewhere in the addition of the multithreaded code that was added quite a while ago. This was added in version 2 and main.

Now that I have my dev setup working I'll continue to track this bug down, but it may take a week or two with my current workload at my day job. Hopefully a fix will be coming soon.

junkert avatar May 25 '23 00:05 junkert

Amazing.. thank so much for quick reply.

luizm avatar May 25 '23 00:05 luizm

@luizm I accidentally committed to Master when making the fix tonight. This looks to fix the issue. Can you let me know if you run into any problems?

https://github.com/smithoss/gonymizer/commit/6fb69566e83f042480e1965ae710beb24c66c53a

junkert avatar May 27 '23 01:05 junkert

@junkert it worked. Thank you so much

IMO you can close this issue.

luizm avatar May 29 '23 16:05 luizm

Woo! Great to hear. Thank you for bringing this up @luizm !

junkert avatar Jun 01 '23 20:06 junkert

Oh, almost forgot I have also released this version as v2.0.1 and have tagged it appropriately. I'll add something to the top of the Issues to let others know that 2.0.0 is broken with UUIDs.

junkert avatar Jun 01 '23 20:06 junkert