Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-14557

Improve row layout.

Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • None
    • None
    • Docs Required, Release Notes Required

    Description

      Motivation.

      When one try to read a column value from a row, the very first check will be a null-check.
      As Null-Table resides right after an Offset-Table, the we need 2 jumps to for the null-check.

      Description.

      Assuming, Null-Table reserves a bit for each columns even if the columns is not Nullable,
      Null-table has constant size (within same version of schema) and we no need extra bytes to persist it's length into the tuple.

      • Null-checks will not require extra read for Offset-Table size and extra jump.
      • Offset-Table will not need extra read/jump to reach as Null-Table size is constant.

      Let's just swap these tables in layout and fix docs README.md and IEP.

       

      Attachments

        Issue Links

        Activity

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

          People

            amashenkov Andrey Mashenkov Assign to me
            amashenkov Andrey Mashenkov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0h
              0h
              Logged:
              Time Spent - 10m
              10m

              Slack

                Issue deployment