Shauren

This is completely pointless, that data is still in .pkt files. All WSTC does is write the data as a single binary blob (it will never attempt to parse packets...

> Master runs in utf8mb4 on default. On new installations it does but I might have forgotten to include conversion on database level, only did each table)

Not only thread A still needs to continue reading as many items from packets as it originally intended, older packets must not overwrite stored values from new packets (important for...

My idea was to make all properties on those classes nullable, have each packet operate on new object, store them in ConcurrentPriorityQueue then when time comes to output sql, merge...

This is definitely also broken for old updatefields too - it also uses a List to store dynamic fields

This might be the reason in pre-cata versions - CurrentMapId has ThreadStatic attribute so each thread has different value if you are using multithreaded parsing https://github.com/TrinityCore/WowPacketParser/blob/b223e9fbba2f648baf983ff9b569165d4a6b7c26/WowPacketParser/Parsing/Parsers/MovementHandler.cs#L15-L16

1.13 is using bfa packets, that has mapid in UPDATE_OBJECT, it should output correctly

1 & 4 are probably autoattack or reactstate buttons, you can't put them in this table

It is, 3.3.5 too (well, not ragnaros specifically but other destructibles - LK, ToC)

Yes, it is a wrong approach, creature_template_addon does not apply to pets (you wouldnt want some open world creature with huge buff to retain it after taming)