[ 
https://issues.apache.org/jira/browse/TEZ-1114?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Rajesh Balamohan updated TEZ-1114:
----------------------------------

    Attachment: TEZ-1114-v3.patch

Thanks Sid

- Removed unused imports and variables
- In scattergather shuffle, SSL factory cleanup is moved to Shuffle (where the 
fetcher threads are created).  Also, HttpConnectionParam is created only once 
here.
- Added synchronization to HttpConnectionParamsBuilder.setSSL().  
cleanupSSLFactory() was added as a defensive mechanism.  
Since SSLFactory is static, it is possible that some thread had initialized it 
and the next wants to disable SSL. Though it is not possible with the current 
codebase, i added it as a defensive mechanism.
- Removed HttpConnectionParams from ShuffleManager.constructFetcherForHost()

[~hitesh] This patch does not have the unit test.  I will add it asap.

> Fix encrypted shuffle
> ---------------------
>
>                 Key: TEZ-1114
>                 URL: https://issues.apache.org/jira/browse/TEZ-1114
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Hitesh Shah
>            Assignee: Rajesh Balamohan
>         Attachments: TEZ-1114-v1.patch, TEZ-1114-v2.patch, TEZ-1114-v3.patch
>
>
> The failure should be detected early by the fetcher and a better error 
> message generated denoting that ssl is not supported for now. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to