SEZ9 commented on PR #12048: URL: https://github.com/apache/seatunnel/pull/12048#issuecomment-5674352347
Thanks @goutamadwant for the update in e5ffcd0f1. **F2 (`ServicesResourceTransformer`):** Good to know the root POM supplies it. Since your check covered the relocated Jackson providers, could you also confirm that the service provider entries for the relocated `com.azure`, `io.netty`, and `org.apache.qpid.proton` packages are present and rewritten in the shaded JAR? With that I'm happy to mark F2 resolved. **F1 (retention-trimmed checkpoint):** As stated in my earlier round (5088543092), the fail-fast behaviour is a deliberate design choice, not a defect. I consider F1 closed. **F3 (connection-string masking):** Closed based on the verification described earlier in this thread and re-confirmed in the 9377e822c round. **F4–F8:** I don't see these addressed yet. Could you reply per item — pinned transitive Netty/Jackson/Proton versions, hub-scoped `EntityPath` SAS policies, `prefetch_count >= max_batch_size` validation at job submission, partition rediscovery, and reconnect/retry tuning — or note which you'd prefer to defer to a follow-up with a documented caveat? On CI: as you noted, the Event Hubs emulator E2E was skipped in the previous run and still needs confirmation. Please ping once a fresh run has actually executed it. <!-- streview-comment:1065 --> -- 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]
