[13] RFR: CONFIG level logging statements printed in CLDRCalendarDataProviderImpl.java even when default log Level is INFO

[Nishit Jain]

Hi Naoto,

Thanks for the explanation. Change looks fine to me.

Regards,
Nishit Jain
On 19-02-2019 22:51, Naoto Sato wrote:
> Hi Nishit,
>
> The reason is that "US" is the only required locale in the JDK (cf. 
> Locale.getAvailableLocales(). In fact, initially I supplied "001" with 
> it, as it means the "world" in CLDR, but it broke some existing tests. 
> "001" returns "MONDAY" as the first day of week, whereas "SUNDAY" in 
> US. For the compatibility reason, I had to resort to "US". I am not 
> sure we want to hardcode "1" in this case without any convincing reason.
>
> Naoto
>
> On 2/19/19 6:37 AM, Nishit Jain wrote:
>> Hi Naoto,
>>
>> Why is the default region set to "US" if there is no region specified 
>> in the locale? is this the default behavior of "first day of week" 
>> and "minimal days in first week" when a region is missing or the 
>> default behavior is that it returns "1"? Can't we just return "1" 
>> instead of setting the region to "US"?
>>
>> Regards,
>> Nishit Jain
>> On 16-02-2019 04:25, Naoto Sato wrote:
>>> Hello,
>>>
>>> Please review the fix to the following issue:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8218960
>>>
>>> The proposed changeset is located at:
>>>
>>> http://cr.openjdk.java.net/~naoto/8218960/webrev.00/
>>>
>>> The CONFIG message was generated because 
>>> CLDRCalendarDataProviderImpl was returning null for locales without 
>>> region. Use "US" as the default region in such a case.
>>>
>>> Naoto
>>