Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-20893

BloomK Filter probing method is not thread safe

Log workAgile BoardRank to TopRank to BottomBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • None
    • storage-api
    • None

    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.

       

      Attachments

        1. HIVE-20893.patch
          51 kB
          Slim Bouguerra
        2. HIVE-20893.WIP.patch
          2 kB
          Gopal Vijayaraghavan

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            bslim Slim Bouguerra Assign to me
            bslim Slim Bouguerra
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment