[jira] [Created] (ARROW-14123) [C++] DayTimeIntervalType::DayMilliseconds comparison function seems incorrect

["Phillip Cloud (Jira)" <ji...@apache.org>]

Phillip Cloud created ARROW-14123:
-------------------------------------

             Summary: [C++] DayTimeIntervalType::DayMilliseconds comparison function seems incorrect
                 Key: ARROW-14123
                 URL: https://issues.apache.org/jira/browse/ARROW-14123
             Project: Apache Arrow
          Issue Type: Bug
          Components: C++
            Reporter: Phillip Cloud
             Fix For: 6.0.0


The less than operator ({{operator<}}) implemented on {{DayTimeIntervalType::DayMilliseconds}} seems incorrect:

{code:cpp}
    bool operator<(DayMilliseconds other) const {
      return this->days < other.days || this->milliseconds < other.milliseconds;
    }

{code}

With this implementation, an example (in pseudocode) such as 

{code:cpp}
DayMilliseconds { days = 10, milliseconds = 999 } < DayMilliseconds { days = 9, milliseconds = 1000 }
{code}

would return {{true}} when it should return {{false}}.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)