[BUG-python/deployment] Create resources with custom `UUID`
davidberenstein1957 opened this issue · 4 comments
Describe the bug
However, the newly created object seems to have its own UUID, not the one I used to initialize. Am I doing something wrong?
Stacktrace and Code to create the bug
Expected behavior
I would expect it to pick up on my custom UUID
Environment:
- Argilla Version [e.g. 1.0.0]: 2.0.1
- ElasticSearch Version [e.g. 7.10.2]:
- Docker Image (optional) [e.g. argilla:v1.0.0]:
Additional context
@frascuchon @jfcalvo should we align the id
behaviour with the Record
class and start differentiating between id
and _server_id
? WDYT?
@davidberenstein1957 is not the Record
's external_id
solving the problem of having a user custom id?
@jfcalvo, I wouldn't say so. A record could have an external ID but I assumed other resources could have too, since there was a user that intended to create a User
with a custom UUID
. So, if we don't want to allow for "Create resources with custom UUID
", perhaps we can allow people to pass an external_id
, in a same way as we handle that for Record
s
@jfcalvo, I wouldn't say so. A record could have an external ID but I assumed other resources could have too, since there was a user that intended to create a
User
with a customUUID
. So, if we don't want to allow for "Create resources with customUUID
", perhaps we can allow people to pass anexternal_id
, in a same way as we handle that forRecord
s
Yes, I think one thing is our internal database id and another one allowing users to use custom id's for reference. If it's really necessary we can add external_id
to other resources.
Another alternative solution to this is adding a metadata
JSON column to all our resources so users can attach custom ids or other information that they need.
cc @frascuchon because we have been discussing this metadata
feature other times in the past and maybe we can take the change of this issue to do it.