Michal Ostrowski has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11535


Change subject: IMPALA-6661 Consider NaN values equal for grouping purposes.
......................................................................

IMPALA-6661 Consider NaN values equal for grouping purposes.

Similar to the treatment of NULLs, we want to consider NaN values as equal
when grouping.

- When detecting a NaN in a set of row values, the NaN value must be converted
  to a canonical value - so that all NaN values have the same bit-pattern for
  hashing purposes.

- When doing equality evaluation, floating point types must have additional
  logic to consider NaN values as equal.

- Existing logic for handling NULLs in this way is appropriate for triggering
  this behavior for NaN values.

- Relabel "force null equality" as "inclusive equality" to expand the scope
  of the concept to a more generic form that includes NaN.

Change-Id: I996c4a2e1934fd887046ed0c55457b7285375086
---
M be/src/codegen/codegen-anyval.cc
M be/src/codegen/codegen-anyval.h
M be/src/exec/hash-table.cc
M be/src/exec/hash-table.h
M be/src/exec/hash-table.inline.h
M be/src/runtime/raw-value.cc
M be/src/runtime/raw-value.h
M be/src/runtime/raw-value.inline.h
M testdata/workloads/functional-query/queries/QueryTest/exprs.test
9 files changed, 150 insertions(+), 29 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/35/11535/1
--
To view, visit http://gerrit.cloudera.org:8080/11535
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I996c4a2e1934fd887046ed0c55457b7285375086
Gerrit-Change-Number: 11535
Gerrit-PatchSet: 1
Gerrit-Owner: Michal Ostrowski <[email protected]>

Reply via email to