kafka-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ismael Juma (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (KAFKA-4922) Fix several FindBugs warnings in Clients and Connect
Date Fri, 26 May 2017 12:44:04 GMT

     [ https://issues.apache.org/jira/browse/KAFKA-4922?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Ismael Juma updated KAFKA-4922:
-------------------------------
    Resolution: Fixed
        Status: Resolved  (was: Patch Available)

I believe these were fixed via a different PR.

> Fix several FindBugs warnings in Clients and Connect
> ----------------------------------------------------
>
>                 Key: KAFKA-4922
>                 URL: https://issues.apache.org/jira/browse/KAFKA-4922
>             Project: Kafka
>          Issue Type: Improvement
>          Components: clients, KafkaConnect
>    Affects Versions: 0.10.2.0
>         Environment: Identified by FindBugs, non-software and -platform specific
>            Reporter: Daan Rennings
>            Priority: Minor
>              Labels: newbie
>             Fix For: 0.11.0.0
>
>         Attachments: ClientsFindBugsReport.html, ConnectAPIFindBugsReport.html, ConnectRuntimeFindBugsReport.html
>
>
> Four easy to fix warnings (not a complete set of the current FindBugs warnings) as identified
by FindBugs and stated in the attached reports:
> -org.apache.kafka.common.utils.Bytes.equals(Object) does not check for null argument
(Low priority, Bad Practice)
> -Primitive boxed just to call toString in org.apache.kafka.connect.data.Decimal.builder(int)
(High Priority, Performance)
> -Primitive boxed just to call toString in new org.apache.kafka.connect.runtime.Worker(String,
Time, ConnectorFactory, WorkerConfig, OffsetBackingStore) (High Priority, Performance)
> -Primitive is boxed to call Integer.compareTo(Integer): use Integer.compare(int, int)
instead (High Priority, Performance)



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message