kriti-sc opened a new pull request, #2771:
URL: https://github.com/apache/iggy/pull/2771

   Closes #2770 
   
   ## Rationale
   The sink_connector! and source_connector! macros export a generic open 
symbol, which collides with POSIX open() in a cdylib. As a result, calls to 
open() resolve to the plugin’s FFI function instead of libc, causing argument 
misinterpretation and a segfault.
   
   ## What changed?
   Renamed the exported FFI symbols to avoid collision with libc/POSIX names:
   
   open → iggy_sink_open / iggy_source_open
   close → iggy_sink_close / iggy_source_close
   consume → iggy_sink_consume
   handle → iggy_source_handle
   version → iggy_sink_version / iggy_source_version
   
   
   ## Local Execution
   
   - Passed - Originally implemented for the Delta Sink Connector. This change 
fixed the segfault.
   
   ## AI Usage
   
   If AI tools were used, please answer:
   1. Which tools? Claude Code
   2. Scope of usage? Autocomplete
   3. How did you verify the generated code works correctly?  The Delta Sink 
for which this change was originally implemented stopped giving the 
segmentation error and ran successfully
   4. Can you explain every line of the code if asked? Yes
   
   -->
   


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