Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
RediStack
RediStack
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 18
    • Issues 18
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
    • Iterations
  • Merge Requests 3
    • Merge Requests 3
  • Requirements
    • Requirements
    • List
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Security & Compliance
    • Security & Compliance
    • Dependency List
    • License Compliance
  • Operations
    • Operations
    • Incidents
    • Environments
  • Analytics
    • Analytics
    • CI / CD
    • Code Review
    • Insights
    • Issue
    • Repository
    • Value Stream
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Nathan Harris
  • RediStackRediStack
  • Issues
  • #47

Closed
Open
Opened Jun 12, 2019 by Nathan Harris@MordilMaintainer6 of 8 tasks completed6/8 tasks

SSWG Review Feedback

During the feedback thread for reviewing the SSWG-0004 RedisNIO proposal, several action items were identified to make RedisNIO the best it can be for 1.0.0.

  • Change RedisCommandHandler from open to public final
  • Move & Rename Redis.makeConnection to RedisConnection.connect
  • Move & Rename Redis.makeDefaultClientBootstrap to ClientBootstrap.makeRedisTCPClient
  • Remove Redis namespace enum
  • Add label to RESPValueConvertible.init requirement
  • Change RESPTranslator logic to copy the ByteBuffer and move the reader index instead of a mutable position value.
  • Clarify language in RESPClient.expire to be consistent with timeouts instead of deadlines
  • Rename RedisCommandContext to RedisCommand
    • This should include documentation to warn users of reference semantics from the EventLoopPromise
Edited Jul 01, 2019 by Nathan Harris
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
1.0.0
Milestone
1.0.0 (Past due)
Assign milestone
Time tracking
None
Due date
None
Reference: Mordil/RediStack#47