Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JDK9EA has buggy locale support when running with security manager #10984

Closed
s1monw opened this issue May 5, 2015 · 6 comments
Closed

JDK9EA has buggy locale support when running with security manager #10984

s1monw opened this issue May 5, 2015 · 6 comments
Labels
>bug :Core/Infra/Core Core issues without another label

Comments

@s1monw
Copy link
Contributor

s1monw commented May 5, 2015

this test fails with security manager enabled since yesterday

mvn test -Pdev -Dtests.seed=2DCB147CE811B0B6 -Dtests.class=org.elasticsearch.index.mapper.date.SimpleDateMappingTests -Dtests.method="testLocale" -Des.logger.level=DEBUG -Des.node.mode=network -Dtests.security.manager=true -Dtests.nightly=false -Dtests.heap.size=1024m -Dtests.jvm.argline="-server" -Dtests.locale=es_AR -Dtests.timezone=Atlantic/Faeroe

likely related to changes pushed yesterady maybe this one #10965 @rmuir @rjernst can you take a look...

here is a CI failure for this http://build-us-00.elastic.co/job/es_core_master_metal/9112/

@s1monw s1monw added the >bug label May 5, 2015
@rmuir
Copy link
Contributor

rmuir commented May 5, 2015

if it only happens on JDK9EA, its likely just a bug in that early access release.

@rmuir rmuir changed the title security manager breaks locale support on JDK9 JDK9EA has buggy locale support when running with security manager May 5, 2015
s1monw added a commit that referenced this issue May 5, 2015
@s1monw
Copy link
Contributor Author

s1monw commented May 5, 2015

I pushed a assume for those tests

@dakrone
Copy link
Member

dakrone commented Sep 27, 2016

There have been a lot of JDK 9 EA releases since this was pushed, we should check to see if the assumeFalse can be removed so I'm marking this as adoptme

@bleskes
Copy link
Contributor

bleskes commented Dec 6, 2016

@dakrone I see you added a blocker label. Was that intended?

@dakrone
Copy link
Member

dakrone commented Dec 6, 2016

@bleskes whoops I don't think so, I'll remove it.

@colings86 colings86 added the :Core/Infra/Core Core issues without another label label Mar 21, 2017
@s1monw
Copy link
Contributor Author

s1monw commented Dec 20, 2017

@dakrone I retested and open a joda time issue JodaOrg/joda-time#462

@s1monw s1monw removed the jvm bug label Dec 20, 2017
s1monw added a commit to s1monw/elasticsearch that referenced this issue Jan 4, 2018
Java 9 added some enhancements to the internationalization support that
impact our date parsing support. To ensure flawless BWC and consistent
behavior going forward Java 9 runtimes requrie the system property
`java.locale.providers=COMPAT` to be set.

Closes elastic#10984
s1monw added a commit that referenced this issue Jan 4, 2018
Java 9 added some enhancements to the internationalization support that
impact our date parsing support. To ensure flawless BWC and consistent
behavior going forward Java 9 runtimes requrie the system property
`java.locale.providers=COMPAT` to be set.

Closes #10984
s1monw added a commit that referenced this issue Jan 4, 2018
Java 9 added some enhancements to the internationalization support that
impact our date parsing support. To ensure flawless BWC and consistent
behavior going forward Java 9 runtimes requrie the system property
`java.locale.providers=COMPAT` to be set.

Closes #10984
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Core/Infra/Core Core issues without another label
Projects
None yet
Development

No branches or pull requests

7 participants