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.

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

[FLINK-3382] Improve clarity of object reuse in ReusingMutableToRegularIteratorWrapper #1614

Conversation

greghogan
Copy link
Contributor

No description provided.

@ggevay
Copy link
Contributor

ggevay commented Feb 9, 2016

Shouldn't the current record remain valid if hasNext() returned true? I mean the user might be holding on to the object returned in next, and expect it to not be changed by a hasNext call:

T cur = it.next();
if(it.hasNext()) {
  // here, I would expect cur to not have changed since the next() call
}

@greghogan
Copy link
Contributor Author

Ah, yes, now I see. I'll just burn this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants