yx9o commented on PR #9397: URL: https://github.com/apache/rocketmq/pull/9397#issuecomment-2919525830
> 1、Could you provide the performance benchmark data for the original Fastjson 1? 2、Could you provide compatibility test results with Fastjson 1, particularly when the object contains camel-case properties? Thank you for your review. The data for the two questions are as follows: 1. <img width="1633" alt="image" src="https://github.com/user-attachments/assets/23fb4fa0-c383-4a41-9a14-a5337d1b7726"; /> 2. <img width="852" alt="image" src="https://github.com/user-attachments/assets/19109938-7a1b-4b3c-bbdf-00ed199a86f2"; /> <img width="938" alt="image" src="https://github.com/user-attachments/assets/da117e3b-a15a-4c1c-ae31-8b7aeda2b589"; /> <img width="676" alt="image" src="https://github.com/user-attachments/assets/34a63e07-a9b0-4659-b95f-1811042e0048"; /> Regarding question 2, I encountered the camel case problem of getCId during the replacement test. The solution is to move the @JSONField(name = "c") annotation from the attribute to the get/set method. All the replaced codes are covered by unit tests. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
