bharathgunapati opened a new pull request, #57:
URL: https://github.com/apache/flink-connector-http/pull/57

   ## What is the purpose of the change
   
   After FLINK-40277, clarify that when the HTTP sink submits a batched 
request, one HTTP response applies to every request entry in that batch: a 
retryable response retries the whole batch, and a fatal / exhausted response 
fails the batch as a unit.
   
   ## Brief change log
   
   - Document batch-level retry / failure semantics for Table and DataStream 
docs (en + zh).
   - Clarify `SinkHttpClientResponses` and writer error messaging around 
request entries.
   - Add tests covering batch classification and exhausted retryable batch 
failures.
   
   ## Compatibility note
   
   - Behaviour aligns with the existing batch submitter (one response for the 
submitted batch). Messaging and docs make that explicit.
   - Follow-up from the FLINK-40275 umbrella after #54.
   
   ## Verifying this change
   
   `mvn -pl flink-connector-http -DskipITs -Dskip.archunit=true 
-Dspotless.check.skip=true -Dcheckstyle.skip=true 
-Dtest=HttpSinkWriterTest,BatchRequestSubmitterTest,JavaNetSinkHttpClientTest 
test`
   
   ## Documentation
   
   - Docs updated for sink batch retry semantics.


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