You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@iggy.apache.org by "mattp5657 (via GitHub)" <gi...@apache.org> on 2026/10/02 15:29:47 UTC

Re: [PR] perf(connectors): drain field_mappings instead of clone-then-clear [iggy]

mattp5657 commented on PR #4362:
URL: https://github.com/apache/iggy/pull/4362#issuecomment-5955676105

   @hubcio For these warnings I kept parity with master to keep this PR single concern. 
   
   I opened #4386 to cover them, including the proto reverse lookup. 
   
   A runtime check added complexity and gave false positives (simd_json keeps duplicate keys when parsing), so I'm proposing a config check at load time plus a lighter payload safeguard. Happy to discuss on the issue.


-- 
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: commits-unsubscribe@iggy.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org