DanielLeens commented on issue #12102: URL: https://github.com/apache/seatunnel/issues/12102#issuecomment-5559104664
Thanks @CryoThrust. The revised STIP now has the single Phase 1 contract that was missing. I independently checked the linked Mem0 reference: it documents POST /v3/memories/add/, Authorization: Token authentication, at least one of user_id, agent_id, app_id, or run_id, and asynchronous tracking through an event_id. The first implementation may therefore proceed as a hosted Mem0 Platform V3 ADD-only connector. Please make Content-Type: application/json and Accept: application/json required request headers and mock-server assertions. For the default asynchronous flow, acceptance must require the documented success response with a nonblank event_id; a bare successful status without the contract payload must not be acknowledged as accepted delivery. Keep the provider adapter and its status/response classification local to this connector. Current connector-http-base retries I/O exceptions and returns HTTP status responses, but it does not provide the V3-specific event contract or retry classification. Preserve the stated at-least-once accepted-delivery boundary, no DELETE, no generic compatible promise, and no OSS profile in this PR. The test matrix should cover the exact path and headers, scope validation, malformed or missing event_id responses, terminal authentication and request failures, retriable timeout, rate-limit, and server failures, plus redacted diagnostics. -- 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]
