Clojure

Primitive vectors should use new hash

Details

  • Type: Enhancement Enhancement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: Release 1.6
  • Fix Version/s: Release 1.6
  • Component/s: None
  • Labels:
    None
  • Patch:
    Code and Test
  • Approval:
    Ok

Description

Primitive vectors created via vector-of still use Java hashCode for hasheq.

Solution: Make primitive vectors implement IHashEq and call Murmur3.hashOrdered().

Patch: clj-1331-v2.patch
Depends on: CLJ-1338 (must be applied first)

  1. clj-1331-v1.diff
    29/Jan/14 6:03 PM
    2 kB
    Andy Fingerhut
  2. clj-1331-v2.patch
    04/Feb/14 5:05 PM
    2 kB
    Alex Miller

Activity

Hide
Andy Fingerhut added a comment -

Patch clj-1331-v1.diff is one way to change primitive vectors to use Murmur3 hash.

Show
Andy Fingerhut added a comment - Patch clj-1331-v1.diff is one way to change primitive vectors to use Murmur3 hash.

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: