You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "PJ Fanning (Jira)" <ji...@apache.org> on 2022/09/08 10:20:00 UTC

[jira] [Created] (DRILL-8302) tidy up some char conversions

PJ Fanning created DRILL-8302:
---------------------------------

             Summary: tidy up some char conversions
                 Key: DRILL-8302
                 URL: https://issues.apache.org/jira/browse/DRILL-8302
             Project: Apache Drill
          Issue Type: Improvement
            Reporter: PJ Fanning


As part of DRILL-8301, I spotted code that could be tidied up. The aim of this issue is to reduce the size of DRILL-8301 without introducing changes to the char encodings.
 * uses of a pattern like `new String("aaaa")` - IntelliJ and other tools highlight this as unnecessary
 * uses of `new String(bytes, StandardCharsets.UTF_8.name())` - better to use `new String(bytes, StandardCharsets.UTF_8)`
 * use Base64 encodeToString instead of case where we encode to bytes and then do our own encoding of those bytes to a String
 * Replace existing code with `Charset.forName("UTF-8")` to use `StandardCharsets.UTF_8`



--
This message was sent by Atlassian Jira
(v8.20.10#820010)