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

Row format. Use varint for sizes.

Attach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Trivial
    • Resolution: Unresolved
    • None
    • None
    • None
    • Docs Required, Release Notes Required

    Description

      We already compact row chunks after writing it and can ensure the row is always written in the most compact way.

      It may be worth writing chunk length as Varint. At least for tiny chunks.

      Saving just 1-2 bytes for mid-size rows (payload of 255->64k) may look not too beneficial,
      but 3-6 bytes for tiny rows does (with payload size up to 255 bytes). E.g. when the key is Long.

      Attachments

        Issue Links

        Activity

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

          People

            Unassigned Unassigned
            amashenkov Andrey Mashenkov

            Dates

              Created:
              Updated:

              Slack

                Issue deployment