Reranko05 opened a new pull request, #51253:
URL: https://github.com/apache/arrow/pull/51253

   ### Rationale for this change
   
   This revisits the implementation based on the review feedback from 
https://github.com/apache/arrow/pull/50905#issuecomment-5393735829 and uses 
Arrow's existing `JsonObjectParser` helper to provide a higher-level interface 
over simdjson.
   
   ### Changes
   
   - Replace manual simdjson On-Demand parsing in `OpaqueType::Deserialize()` 
with `JsonObjectParser`.
   - Use `JsonObjectParser::GetString()` to retrieve the required `type_name` 
and `vendor_name` fields.
   - Remove manual JSON object iteration, key handling, and explicit JSON type 
checks.
   - Update `OpaqueType::Deserialize` tests to verify invalid input is rejected 
without relying on exact error messages.
   
   Fixes: #51252


-- 
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]

Reply via email to