[jira] [Commented] (AVRO-2350) Use Switch Statement for Strings in LogicalTypes

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/AVRO-2350?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16790744#comment-16790744 ] 

ASF subversion and git services commented on AVRO-2350:
-------------------------------------------------------

Commit 63e02101cf94d8ab0e1ae01baafe04c5767aa371 in avro's branch refs/heads/master from Beluga Behr
[ https://gitbox.apache.org/repos/asf?p=avro.git;h=63e0210 ]

AVRO-2350: Use Switch Statement for Strings in LogicalTypes


> Use Switch Statement for Strings in LogicalTypes
> ------------------------------------------------
>
>                 Key: AVRO-2350
>                 URL: https://issues.apache.org/jira/browse/AVRO-2350
>             Project: Apache Avro
>          Issue Type: Improvement
>          Components: java
>    Affects Versions: 1.9.0
>            Reporter: David Mollitor
>            Assignee: David Mollitor
>            Priority: Minor
>
> https://github.com/apache/avro/blob/master/lang/java/avro/src/main/java/org/apache/avro/LogicalTypes.java#L59-L101
> {quote}
> The switch statement compares the String object in its expression with the expressions associated with each case label as if it were using the String.equals method; consequently, the comparison of String objects in switch statements is case sensitive. The Java compiler generates generally more efficient bytecode from switch statements that use String objects than from chained if-then-else statements.
> https://docs.oracle.com/javase/7/docs/technotes/guides/language/strings-switch.html
> {quote}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)