[ 
https://issues.apache.org/jira/browse/GIRAPH-930?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14076462#comment-14076462
 ] 

Sergey Edunov commented on GIRAPH-930:
--------------------------------------

Hi Mostafa, 
Thank you for working on this. Your implementation looks good to me, but I was 
thinking about maybe cleaning up some code above instead, what do you think? 
Attaching another patch (haven't tested yet, will do in the afternoon) 


> Trailing space in ZooKeeper serverList file trips some file systems
> -------------------------------------------------------------------
>
>                 Key: GIRAPH-930
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-930
>             Project: Giraph
>          Issue Type: Bug
>         Environment: HDInsight
>            Reporter: Mostafa Elhemali
>         Attachments: GIRAPH-930.diff, GIRAPH-930.patch
>
>
> In Azure HDInsight (Hadoop in Microsoft Azure), the default file system is 
> the WASB file system which places data on Azure blob storage. This file 
> system doesn't handle trailing spaces that well, so when the ZooKeeperManager 
> class tries to create the serverList file with a trailing space in there 
> things fall apart.
> Ideally WASB would handle trailing spaces, but in this case the trailing 
> space is not really necessary. Can we remove it?



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

Reply via email to