You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-dev@jackrabbit.apache.org by Chetan Mehrotra <ch...@gmail.com> on 2014/05/26 13:56:59 UTC

Re: svn commit: r1597569 - in /jackrabbit/oak/trunk/oak-core/src: main/java/org/apache/jackrabbit/oak/query/ast/LowerCaseImpl.java test/java/org/apache/jackrabbit/oak/query/index/TraversingIndexQueryTest.java

On Mon, May 26, 2014 at 5:19 PM,  <al...@apache.org> wrote:
> +                        public String apply(String input) {
> +                            return input.toLowerCase();
> +                        }

Do we need to take care of Turkish locale bug (OAK-260) and specify
locale here also?

Chetan Mehrotra

Re: svn commit: r1597569 - in /jackrabbit/oak/trunk/oak-core/src: main/java/org/apache/jackrabbit/oak/query/ast/LowerCaseImpl.java test/java/org/apache/jackrabbit/oak/query/index/TraversingIndexQueryTest.java

Posted by Alex Parvulescu <al...@gmail.com>.

This specific issue is about the IllegalStateException, so I would prefer
to not mix stuff.

I left the TODO about the locale in the code, so when OAK-260 gets fixed,
the fix it will probably get applied to both the old code and the new code.

alex

On Mon, May 26, 2014 at 1:56 PM, Chetan Mehrotra
<ch...@gmail.com>wrote:

> On Mon, May 26, 2014 at 5:19 PM,  <al...@apache.org> wrote:
> > +                        public String apply(String input) {
> > +                            return input.toLowerCase();
> > +                        }
>
> Do we need to take care of Turkish locale bug (OAK-260) and specify
> locale here also?
>
> Chetan Mehrotra
>