[ https://issues.apache.org/jira/browse/SPARK-18800?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sean Owen updated SPARK-18800: ------------------------------ Assignee: Liang-Chi Hsieh Priority: Minor (was: Major) > Correct the assert in UnsafeKVExternalSorter which ensures array size > --------------------------------------------------------------------- > > Key: SPARK-18800 > URL: https://issues.apache.org/jira/browse/SPARK-18800 > Project: Spark > Issue Type: Improvement > Components: SQL > Reporter: Liang-Chi Hsieh > Assignee: Liang-Chi Hsieh > Priority: Minor > Fix For: 2.2.0 > > > UnsafeKVExternalSorter uses UnsafeInMemorySorter to sort the records of > BytesToBytesMap if it is given a map. > Currently we use the number of keys in BytesToBytesMap to determine if the > array used for sort is enough or not. We has an assert that ensures the size > of the array is enough: map.numKeys() <= map.getArray().size() / 2. > However, each record in the map takes two entries in the array, one is record > pointer, another is key prefix. So the correct assert should be map.numKeys() > * 2 <= map.getArray().size() / 2. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org