[HIVE-20893] BloomK Filter probing method is not thread safe

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Component/s: storage-api
• Assignee: Slim Bouguerra
• Reporter: Slim Bouguerra
• Created: 08/Nov/18 18:15
• Updated: 11/Nov/18 21:44
• Resolved: 11/Nov/18 21:44

Description

As far i can tell this is not an issue for Hive yet (most of the usage of probing seems to be done by one thread at a time) but it is an issue of other users like Druid as per the following issue.https://github.com/apache/incubator-druid/issues/6546

The fix is proposed by the author of https://github.com/apache/incubator-druid/pull/6584 is to make couple of local fields as ThreadLocals.

Idea looks good to me and doesn't have any perf drawbacks.