search

oap-project / gazelle_plugin

Native SQL Engine plugin for Spark SQL with vectorized SIMD optimizations.
Apache License 2.0
256 stars 77 forks source link

[NSE-1125] Add status check for hashing GetOrInsert #1126

Closed jackylee-ch closed 1 year ago

jackylee-ch commented 1 year ago

What changes were proposed in this pull request?

Index returns -1 when HashMap.GetOrInsert failed with some problems such as buffer overflow or OOM. And it whould finally core dump when trying to use this index. Here we add a status check and throw exception if it failed.

How was this patch tested?

unit tests.

github-actions[bot] commented 1 year ago

https://github.com/oap-project/native-sql-engine/issues/1125