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

Remove max-retain and policies #1184

Merged
merged 24 commits into from Jan 8, 2019

Conversation

viliam-durina
Copy link
Contributor

No description provided.

@viliam-durina viliam-durina changed the title [WIP] Remove max-retain and policies Remove max-retain and policies Dec 20, 2018
@devOpsHazelcast
Copy link
Contributor

Test FAILed.

@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 7, 2019
@@ -190,6 +185,18 @@ static String uniqueName(
}
}

/**
* Returns new instance with emit policy replaced with the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Returns a new instance

/**
* Special-case implementation for zero inputs (source processors).
*/
private static final class SingleInputImpl extends WatermarkCoalescer {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it zero inputs or single input? Either Javadoc or class name is wrong.

@hazelcast hazelcast deleted a comment from devOpsHazelcast Jan 8, 2019
@viliam-durina viliam-durina merged commit 63f67ed into hazelcast:master Jan 8, 2019
@viliam-durina viliam-durina deleted the drop-max-retain branch January 8, 2019 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants