Ah, nice catch.


On Fri, Mar 14, 2014 at 1:22 PM, Chris Ngan <notifications@github.com>wrote:

> In lib/mortar/s3.rb:
>
> > +        rescue
> > +          error("It is strongly suggested that your bucket name does not contain an underscore character.\nPlease see http://blog.mortardata.com/post/58920122308/s3-hadoop-performance
>  at tip #4.")
> > +        end
> > +
> > +       return uri.host,uri.path[1, uri.path.length]
> > +      end
> > +
> > +
> > +      # checks if string is a valid s3 path
> > +      def is_valid_s3_path(s3_path)
> > +        return s3_path[0,5] == "s3://" && s3_path[5,s3_path.length].index("/")
> > +      end
> > +
> > +
> > +      def remove_slash(str)
> > +        if str[str.length-1, str.length] == "/"
>
> Oh it doesn't work in 1.8.7. I remember that's why I used it that way. yay
> ruby...
> 1.8.7 :001 > "asef"[-1]
> => 102
> 1.8.7 :002 > "asef"[-1,1]
> => "f"
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/mortardata/mortar/pull/184/files#r10617385>
> .
>



-- 
Doug Daniels / CTO
MORTAR DATA / www.mortardata.com
t: @ddaniels888