Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-6580

Remove Locale.setDefault

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 1.38.0
    • None

    Description

      Remove all calls to Locale.setDefault in tests and production code.

      That function does not operate on the current thread; it affects all threads in the JVM. As such, it may affect other tests running at the same time, and affect other statements running at the same time.

      I hope, and believe, that the production code does not depend on Locale.getDefault. But let's make sure by removing setDefault from all tests.

      Add Locale.setDefault to forbidden-apis.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              julianhyde Julian Hyde
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: