Uploaded image for project: 'Ratis'
  1. Ratis
  2. RATIS-2165

Do not include RaftClientRequest Message in ReadIndexRequest

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

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 3.2.0, 3.1.2
    • Linearizable Read
    • None

    Description

      It seems that in RaftServerImpl.sendReadIndexAsync, we do not need to serialize the message of the request, because we only use request.getClientId ().

      For this situation, we can provide a setMessage interface that is set to null every time sendReadAsync is called.
      Or we can regenerate a RaftClientRequest without Message.
       
      Whether this has good returns can be further discussed

      Attachments

        Activity

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

          People

            tohsakarin__ yuuka
            tohsakarin__ yuuka
            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 - 20m
                20m

                Slack

                  Issue deployment