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

Wrong argument in the WorkloadManager.resetAndQueryKill()

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: Closed
    • Trivial
    • Resolution: Fixed
    • None
    • 3.0.0
    • None
    • None

    Description

      Caused by HIVE-18088, Prasanth Jayachandran, could you please check this?

      private void resetAndQueueKill(Map<WmTezSession, KillQueryContext> toKillQuery,
      KillQueryContext killQueryContext, Map<WmTezSession, GetRequest> toReuse) {
      
      WmTezSession toKill = killQueryContext.session;
      ...
      if (poolState != null) {
      poolState.getSessions().remove(toKill);
      poolState.getInitializingSessions().remove(toKill);
      ...
      
      

      getInitializingSessions() returns List of SessionInitContext, so toKill definitely can't be in this list and therefore no needs to remove it.

       

      Attachments

        1. HIVE-18492.patch
          0.9 kB
          Oleg Danilov
        2. HIVE-18492.2.patch
          1 kB
          Oleg Danilov
        3. HIVE-18492.04.patch
          0.9 kB
          Sergey Shelukhin
        4. HIVE-18492.03.patch
          0.9 kB
          Sergey Shelukhin

        Issue Links

        Activity

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

          People

            olegd Oleg Danilov Assign to me
            olegd Oleg Danilov
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment