goutamadwant commented on PR #12048:
URL: https://github.com/apache/seatunnel/pull/12048#issuecomment-5808304036

   Follow-up at head 1aa4ee993 on the remaining F2/F3/F6 evidence:
   
   - F6: 
`AzureEventHubsSourceConfigTest#factoryRulesRejectBlankAndCrossFieldOptions` 
invokes the factory OptionRule and rejects `prefetch_count < max_batch_size` 
before reader construction.
   - F3: this project's Option API has no sensitivity flag. The shared 
ConfigBuilder log-masking test covers `connection_string`; 
`entityPathRejectionAndStringRepresentationsDoNotExposeCredentials` now checks 
a SAS-bearing string against EntityPath errors and both config/split `toString` 
output, including the full connection string.
   - F2: the root shade configuration includes `ServicesResourceTransformer`, 
inherited by this module. I inspected the freshly packaged connector jar: its 
Jackson service entries are relocated; the resolved Azure/Proton jars contain 
no corresponding service descriptors, and Netty is not packaged here as a 
compile dependency. This is packaging evidence, not emulator runtime proof.
   
   The commits since 386d5c1ea did touch relevant files: 765fb71b0 updates 
docs/config/emitter diagnostics, 3e4ee6d33 updates shared config/factory 
validation and E2E assertions, and 1aa4ee993 adds credential/checkpoint test 
assertions. F1/F4/F5/F7/F8 remain the documented initial-scope decisions; no 
reset, rediscovery, or retry controls were added. Focused tests passed, while 
live emulator E2E and the new-head Build remain pending.


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