-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix NaN handling in map_subset #9893
Fix NaN handling in map_subset #9893
Conversation
This pull request was exported from Phabricator. Differential Revision: D57681657 |
✅ Deploy Preview for meta-velox canceled.
|
Summary: Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Differential Revision: D57681657
3863707
to
31f77c9
Compare
This pull request was exported from Phabricator. Differential Revision: D57681657 |
Summary: Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Differential Revision: D57681657
31f77c9
to
4d2b732
Compare
This pull request was exported from Phabricator. Differential Revision: D57681657 |
Summary: Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Differential Revision: D57681657
4d2b732
to
916dffa
Compare
This pull request was exported from Phabricator. Differential Revision: D57681657 |
Summary: Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Differential Revision: D57681657
916dffa
to
a4dbe5e
Compare
This pull request was exported from Phabricator. Differential Revision: D57681657 |
Summary: Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Differential Revision: D57681657
a4dbe5e
to
99d4c01
Compare
This pull request was exported from Phabricator. Differential Revision: D57681657 |
99d4c01
to
3c65353
Compare
Summary: Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Summary of changes: - For primitive type keys: Use a map that employs the correct hashing and equality operators for floating types. - For complex type keys: It uses GenericView as a key that uses BaseVector's hashing and equality operators. Those recursively call SimpleVector's respective functions that were addressed in facebookincubator#9963 Differential Revision: D57681657
This pull request was exported from Phabricator. Differential Revision: D57681657 |
Summary: Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Summary of changes: - For primitive type keys: Use a map that employs the correct hashing and equality operators for floating types. - For complex type keys: It uses GenericView as a key that uses BaseVector's hashing and equality operators. Those recursively call SimpleVector's respective functions that were addressed in facebookincubator#9963 Differential Revision: D57681657
3c65353
to
2d27791
Compare
This pull request was exported from Phabricator. Differential Revision: D57681657 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
This pull request has been merged in a10bd49. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Summary: Pull Request resolved: facebookincubator#9893 Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Summary of changes: - For primitive type keys: Use a map that employs the correct hashing and equality operators for floating types. - For complex type keys: It uses GenericView as a key that uses BaseVector's hashing and equality operators. Those recursively call SimpleVector's respective functions that were addressed in facebookincubator#9963 Reviewed By: spershin Differential Revision: D57681657 fbshipit-source-id: 0ce281f303dd0633e698e4b386627c3831567d92
Summary: Pull Request resolved: facebookincubator#9893 Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Summary of changes: - For primitive type keys: Use a map that employs the correct hashing and equality operators for floating types. - For complex type keys: It uses GenericView as a key that uses BaseVector's hashing and equality operators. Those recursively call SimpleVector's respective functions that were addressed in facebookincubator#9963 Reviewed By: spershin Differential Revision: D57681657 fbshipit-source-id: 0ce281f303dd0633e698e4b386627c3831567d92
Summary: Pull Request resolved: facebookincubator#9893 Ensure NaNs values of any binary representations are treated as equal and can be identified as keys in a map. Summary of changes: - For primitive type keys: Use a map that employs the correct hashing and equality operators for floating types. - For complex type keys: It uses GenericView as a key that uses BaseVector's hashing and equality operators. Those recursively call SimpleVector's respective functions that were addressed in facebookincubator#9963 Reviewed By: spershin Differential Revision: D57681657 fbshipit-source-id: 0ce281f303dd0633e698e4b386627c3831567d92
Summary:
Ensure NaNs values of any binary representations are treated as equal and
can be identified as keys in a map.
Depends on #9893 #9963
Differential Revision: D57681657