GitHub user kotarot opened a pull request:
https://github.com/apache/orc/pull/314
ORC-410: Fix a locale-dependent test in TestCsvReader
## Problem
`testCustomTimestampFormat` in `TestCsvReader` fails in some environments
because the test is locale-dependent.
In this test, we try to parse a DateTime string (such as '21 Mar 2018
12:23:34') with a given timestamp format. The problem is that English month
abbreviations (such as 'Mar') are locale-dependent. When the locale of Java
Virtual Machine is a locale where the language is English (e.g., en_US and
en_GB), this test passes without any problems. However, when the locale of JVM
is a locale where the language is non-English (e.g., ja_JP and zh_CN), the test
fails as follows.
```
[INFO] Running org.apache.orc.tools.convert.TestCsvReader
[ERROR] Tests run: 5, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
0.237 s <<< FAILURE! - in org.apache.orc.tools.convert.TestCsvReader
[ERROR]
testCustomTimestampFormat(org.apache.orc.tools.convert.TestCsvReader) Time
elapsed: 0.143 s <<< ERROR!
org.threeten.bp.format.DateTimeParseException: Text '21 Mar 2018 12:23:34'
could not be parsed at index 3
at
org.apache.orc.tools.convert.TestCsvReader.testCustomTimestampFormat(TestCsvReader.java:189)
```
## Solution
The following two solutions can be considered to fix this problem by
updating the test:
(1) Make this test be locale-independent.
(2) Set the locale to en_US in this test.
(1) is better, but it's not an easy task to construct a DateTime string
which can be successfully parsed in all existing locales.
Thus, I adopt (2) and modify the test.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kotarot/orc ORC-410
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/orc/pull/314.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #314
----
commit 43ae8b80783c2af8e155c2fbbfb724bf86b9a5f2
Author: Kotaro Terada <koterada@...>
Date: 2018-09-27T03:45:31Z
Fix a locale-dependent test in TestCsvReader
----
---
