MiguelAnzoWizeline commented on pull request #15549: URL: https://github.com/apache/beam/pull/15549#issuecomment-971912615
Updated RedisCursor compareTo to differentiate start and end, moved util functions to RedisCursorRangeTrakcer. About the internal ByteKeyRangeTracker, I removed it and moved its logic to the RedisCursorRangeTracker as having the internal ByteKeyRangeTracker presented some problems, mainly that the trySplit method would produce ranges with a starting key that was not a valid cursor returned by redis (because of its next function adding 1 to the current ByteKey) this caused problems on the test case with a large number of keys. RedisCursorRangeTracker maintains the same logic used on ByteKeyRangeTracker with the exception of a modified trySplit method. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
