Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-2803

[C++] Put hashing function into src/arrow/util

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 0.12.0
    • C++

    Description

      See https://github.com/apache/arrow/pull/2220

      We should decide what our default go-to hash function should be (maybe murmur3?) and put it into src/arrow/util

      Attachments

        Activity

          pcmoritz Philipp Moritz created issue -
          wesm Wes McKinney added a comment -

          Marking this as resolved with https://github.com/apache/arrow/commit/eaf8d32e5f292dca0aa5b5508041d5d39406224d. See arrow/util/hash-util.h. xxhash is now moved to arrow/util

          wesm Wes McKinney added a comment - Marking this as resolved with https://github.com/apache/arrow/commit/eaf8d32e5f292dca0aa5b5508041d5d39406224d . See arrow/util/hash-util.h. xxhash is now moved to arrow/util
          wesm Wes McKinney made changes -
          Field Original Value New Value
          Fix Version/s 0.12.0 [ 12343858 ]
          Assignee Antoine Pitrou [ pitrou ]
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          apitrou Antoine Pitrou made changes -
          Component/s C++ [ 12328935 ]
          cml Chris Lambertus made changes -
          Workflow jira [ 13496594 ] Classic - editable closed [ 13997654 ]
          rokm Rok Mihevc added a comment -

          This issue has been migrated to issue #15744 on GitHub. Please see the migration documentation for further details.

          rokm Rok Mihevc added a comment - This issue has been migrated to issue #15744 on GitHub. Please see the migration documentation for further details.
          rokm Rok Mihevc made changes -

          People

            apitrou Antoine Pitrou
            pcmoritz Philipp Moritz
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: