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

Josh Elser commented on ACCUMULO-708:
-------------------------------------

Dave,

An issue with your most recent patch:

{noformat}
Short umask = Short.parseShort(line.trim());
{noformat}

As Keith pointed out, we need base 8, but the default radix used by parseShort 
is base 10. Thus, I get the following incorrectly calculated umask set in the 
config.

{noformat}
umask response: 0022
Umask is: 026
Perm is: 751
{noformat}

Changing that Short.parseShort method call creates the expected output and the 
tests then pass.

{noformat}
Short umask = Short.parseShort(line.trim(), 8);
{noformat}

Output:

{noformat}
umask response: 0022
Umask is: 022
Perm is: 755
{noformat}
                
> Modify ClassLoader to support different applications / multi-tenancy
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-708
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-708
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: start
>            Reporter: Dave Marion
>            Assignee: Dave Marion
>              Labels: classloader
>             Fix For: 1.5.0
>
>         Attachments: ACCUMULO-708-1.patch, ACCUMULO-708-2.patch, 
> HelloWorld.jar
>
>   Original Estimate: 24h
>          Time Spent: 33h
>  Remaining Estimate: 0h
>
> I'd like to expand the current classloader to support loading classes from 
> HDFS and different application contexts. I'll be modifying the ticket as the 
> idea matures.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to