Hello. I was going through the bookbrainz database schema, specially the import related tables which will be used in the future for importing large databases.
2023-03-20 07956, 2023
Meziyum
I was able to identify a relationship between import table which is just like the entity table and author_import_header which acts an intermediatory between the import and author_data.
2023-03-20 07956, 2023
Meziyum
The question which I had was suppose if a large amount of books are imported into the database then how is an editor supposed to edit one of those imported book if there was some kind of error. Since i dont see the author_import_header storing the revision but directly the author_data.Wont it be better if like the normal entities, the import entity
2023-03-20 07957, 2023
Meziyum
headers also have revision which are then related to their author_data?
2023-03-20 07946, 2023
kellnerd joined the channel
2023-03-20 07956, 2023
iamgroot joined the channel
2023-03-20 07911, 2023
iamgroot has quit
2023-03-20 07931, 2023
kellnerd has quit
2023-03-20 07927, 2023
kellnerd joined the channel
2023-03-20 07945, 2023
iamgroot joined the channel
2023-03-20 07930, 2023
iamgroot has quit
2023-03-20 07928, 2023
Meziyum has quit
2023-03-20 07926, 2023
Meziyum joined the channel
2023-03-20 07924, 2023
Meziyum has quit
2023-03-20 07912, 2023
monkey
Meziyum: Hi! I think having revision tracking for the temporary import entities would be overkill. The goal is for users to validate/modify the information before making them into regular entities (or merging it into an existing entity), at which point they enter the normal revision system.
2023-03-20 07937, 2023
Meziyum joined the channel
2023-03-20 07913, 2023
Meziyum has quit
2023-03-20 07920, 2023
monkey
Hello maddy007[m] ! Don't hesitate to post your questions in advance, so that I can respond when I am online. You can then check the chat logs and search for your name, and hopefully have an answer waiting for you :)
I thought there was a timing issue but a race condition seems more likely now: Two different places are trying to call `redisClient.connect()` when they are importing the same shared module, which is leading to a timeout in the end...
2023-03-20 07948, 2023
kellnerd
Actually I could squash these commits, but it's also interesting to see how *not* to do this ;)