-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add setStreamId method to EventTransformation to support transformations for new streams #17
Conversation
@@ -32,34 +25,30 @@ | |||
* @deprecated use DEACTIVATE instead | |||
*/ | |||
@Deprecated | |||
public static final Action ARCHIVE = new Action(false, true, false, false); | |||
public static final Action ARCHIVE = new Action(false, true, false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
20c4985
to
336f8a4
Compare
aef7505
to
8e6a8e1
Compare
// public Optional<UUID> setStreamId(final JsonEnvelope event) { | ||
// return Optional.of(STREAM_ID); | ||
// } | ||
// } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this commented out code be removed?
a4de397
to
611a045
Compare
c7e9027
to
75eb75d
Compare
final EventTransformationStreamIdFilter eventTransformationStreamIdFilter = new EventTransformationStreamIdFilter(); | ||
final Optional<UUID> eventTransformationStreamId = eventTransformationStreamIdFilter.getEventTransformationStreamId(newHashSet(transformation1, transformation2), Arrays.asList(event1, event2)); | ||
|
||
assertFalse(eventTransformationStreamId.isPresent()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this correct? The test name seems to indicate that this would be true
…ons for new streams
70cf7b5
to
a989e19
Compare
No description provided.