You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Liya Fan (Jira)" <ji...@apache.org> on 2019/09/09 13:09:00 UTC

[jira] [Updated] (ARROW-6458) [Java] Remove value boxing/unboxing for ApproxEqualsVisitor

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

Liya Fan updated ARROW-6458:
----------------------------
    Summary: [Java] Remove value boxing/unboxing for ApproxEqualsVisitor  (was: [Java] Improve the performance and code structure for ApproxEqualsVisitor)

> [Java] Remove value boxing/unboxing for ApproxEqualsVisitor
> -----------------------------------------------------------
>
>                 Key: ARROW-6458
>                 URL: https://issues.apache.org/jira/browse/ARROW-6458
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Java
>            Reporter: Liya Fan
>            Assignee: Liya Fan
>            Priority: Minor
>              Labels: pull-request-available
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> As discussed in https://github.com/apache/arrow/pull/5195#issuecomment-526157961, there are some problems with the current ways of comparing floating point vectors, we solve them in this PR:
> 1. there are if statements/duplicated members in ApproxEqualsVisitor, making the code redundant and less clear.
> 2. the comparion of float4 and float8 are based on wrapped objects Float and Double, which may have performance penalty.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)