Log workAgile BoardRank to TopRank to BottomBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersConvert to IssueMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.13.0
    • 0.14.0
    • Transactions
    • None

    Description

      Currently RecordUpdater.update and delete take rowid and original transaction as parameters. These values are already present in the row as part of the new ROW_ID virtual column in HIVE-7513, and thus can be read by the writer from there. And the writer will already have to handle skipping ROW_ID when writing, so it needs to be aware of that column anyone.

      We could instead read the values from ROW__ID and then remove it from the object inspector in FileSinkOperator, but this will be hard in the vectorization case where rows are being dealt with 10k at a time.

      For these reasons it makes more sense to do this work in the writer.

      Attachments

        1. HIVE-7571.WIP.patch
          40 kB
          Alan Gates
        2. HIVE-7571.patch
          20 kB
          Alan Gates
        3. HIVE-7571.2.patch
          24 kB
          Alan Gates

        Issue Links

        Activity

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

          People

            gates Alan Gates Assign to me
            gates Alan Gates
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment