SEZ9 commented on issue #12102: URL: https://github.com/apache/seatunnel/issues/12102#issuecomment-5564956527
Thanks @CryoThrust — the revised STIP-38 with a single hosted Mem0 Platform V3 ADD-only profile is reviewable, and the implementation PR can proceed on that basis. To keep the conditions already stated in this thread in one place for the PR: - `Content-Type: application/json` and `Accept: application/json` are required request headers, asserted by the mock server. - Acceptance requires the documented success response with a nonblank `event_id`; a bare 2xx without that payload must not be acknowledged as accepted delivery. - Provider adapter and status/response classification stay local to the connector, reusing only common HTTP configuration. - Preserve the at-least-once accepted-delivery boundary; no DELETE, no generic compatible promise, and no OSS profile in this PR. - Test matrix: exact path and headers, scope validation, malformed or missing `event_id` responses, terminal authentication and request failures, retriable timeout/rate-limit/server failures, and redacted diagnostics. Looking forward to the PR. <!-- streview-comment:871 --> -- 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]
