mohit7705 opened a new pull request, #1373:
URL: https://github.com/apache/datafusion-ballista/pull/1373

   ## Which issue does this PR close?
   
   Related to #1367
   
   ## Rationale for this change
   
   Currently, client, scheduler, and executor components create transport 
connections
   directly, which makes it difficult to extend or customize transport behavior
   (e.g. adding optional TLS support).
   
   This PR introduces transport-level factory abstractions as a first step 
toward
   supporting optional TLS while keeping plain HTTP as the default.
   
   ## What changes are included in this PR?
   
   - Introduces a new `ballista::core::transport` module
   - Adds `ChannelFactory` and `ServerFactory` traits
   - Provides default factory stubs preserving existing behavior
   - Adds TLS factory stubs to demonstrate future extensibility
   
   ## Are there any user-facing changes?
   
   No. This PR does not change any runtime behavior and does not introduce TLS 
yet.
   
   ## Are there any breaking changes?
   
   No.
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to