Ata İlhan Köktürk test_and_ci last 3 years


 3 Collaborator
Brandon Williams , Štefan Miklošovič , Maxwell Guo

 1 Patch
46b36f23cd865ee905742ff1d3da05fc8a467758

46b36f23cd865ee905742ff1d3da05fc8a467758 | Author: Ata İlhan Köktürk <atailhan2006@gmail.com>
 | 2024-09-29 12:31:34+08:00

    Explicitly localize strings to Locale.US for internal implementation
    
    There is inconsistent usage of toLowerCase/toUpperCase methods in the codebase.
    Somewhere we already use Locale.US, somewhere not. That might cause various issues in runtime
    as shown in CASSANDRA-19953 ticket when an environment Cassandra runs in has different locale from expected.
    In this patch, all such method calls explicitly set their locale to Locale.US. This patch also contains
    a checkstyle rule which fails the compilation if toLowerCase or toUpperCase is used.
    We prefer calls to methods in LocalizeString class which use US locale.
    
    patch by Ata İlhan Köktürk; reviewed by Stefan Miklosovic, Brandon Williams for CASSANDRA-19953
    
    Co-authored-by: Maxwell Guo <cclive1601@gmail.com>
    Co-authored-by: Stefan Miklosovic <smiklosovic@apacheorg>