Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

KAFKA-5144 renamed and added comments to make it clear what's going on #2948

Closed
wants to merge 1 commit into from
Closed

KAFKA-5144 renamed and added comments to make it clear what's going on #2948

wants to merge 1 commit into from

Commits on Apr 30, 2017

  1. KAFKA-5144 rename variables and add comments to make it clear what's … 

    …going on

The descendingSubsequence is a misnomer. The linked list is actually arranged so that the lowest timestamp is first and larger timestamps are added to the end, therefore renamed to ascendingSubsequence.
The minElem variable was also misnamed. It's actually the current maximum element as it's taken from the end of the list.
Added comment to get() to make it clear it's returning the lowest timestamp.
    @mihbor
    mihbor committed Apr 30, 2017
    Configuration menu
    Copy the full SHA
    c8d5f46 View commit details
    Browse the repository at this point in the history