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

ALTER TABLE post-commit listener does not include the transactional listener responses

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
    • Major
    • Resolution: Fixed
    • None
    • 2.4.0, 3.0.0
    • None
    • None

    Description

      in HiveMetaStore, alter_table_core does NOT call transactional listener, and the notification ID corresponding to the alter table event is NOT set in the event parameters.

      + alter_table_core
      	
            try {
              Table oldt = this.get_table_core(dbname, name);
              this.firePreEvent(new PreAlterTableEvent(oldt, newTable, this));
              this.alterHandler.alterTable(this.getMS(), this.wh, dbname, name, newTable, envContext, this);
              success = true;
              if (!this.listeners.isEmpty()) {
                MetaStoreListenerNotifier.notifyEvent(this.listeners, EventType.ALTER_TABLE, new AlterTableEvent(oldt, newTable, true, this), envContext);
              }
            } catch (NoSuchObjectException var12) {
              ex = var12;
              throw new InvalidOperationException(var12.getMessage());
            } catch (Exception var13) {
              ex = var13;
              if (var13 instanceof MetaException) {
                throw (MetaException)var13;
              }
      
              if (var13 instanceof InvalidOperationException) {
                throw (InvalidOperationException)var13;
              }
      
      

      Attachments

        1. HIVE-18783.1.patch
          8 kB
          Sergio Peña
        2. HIVE-18783.2.patch
          10 kB
          Sergio Peña
        3. HIVE-18783.3.patch
          10 kB
          Sergio Peña

        Issue Links

        Activity

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

          People

            spena Sergio Peña Assign to me
            linaataustin Na Li
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment