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)