david-streamlio commented on PR #96: URL: https://github.com/apache/pulsar-connectors/pull/96#issuecomment-4938154257
The one-line production change here — `Value.getAsBlob(...)` → `Value.get(...)` — fixes a real, severe defect filed as #97: with the pinned `aerospike-client-bc:4.5.0`, the blob serializer is compiled out, so `estimateSize()` throws `"Object serializer has been disabled"` on **every** write. The sink was completely non-functional. Verified from the client bytecode. Since the old path never succeeded, no existing data depends on the blob format, so switching to a native bin is safe. Flagging per the #78/#79 precedent in case maintainers prefer this fix as its own PR with the test rebased on top. -- 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]
