-
Notifications
You must be signed in to change notification settings - Fork 628
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
ISPN-4753 Add oldValue, oldMetadata and retry flag to filter and convert... #2919
Conversation
* @author wburns | ||
* @since 7.0 | ||
*/ | ||
public enum EventType { |
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.
This enum has become a bit complicated, I am thinking maybe I should change over to a regular class. WDYT?
@@ -27,20 +27,20 @@ | |||
* Notifies all registered listeners of a CacheEntryCreated event. | |||
*/ | |||
void notifyCacheEntryCreated(K key, V value, boolean pre, | |||
InvocationContext ctx, FlagAffectedCommand command); | |||
InvocationContext ctx, FlagAffectedCommand command, InternalCacheEntry<K, V> previousEntry); |
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.
I am thinking of changing these to instead take an Object and Metadata directly so we don't need to clone the Entry and it should work for pre events easier as we won't have an Entry there either.
469dbed
to
7908ed3
Compare
Fixed the comments I had before. |
18127c3
to
ed59e59
Compare
Fixing a few test failures and will update. |
31674f1
to
68c657c
Compare
…erter for Cluster Listeners. * Added new filter and converter specifically for listeners
Tests should be fixed, had some references that were missed when I rebased and fixed conflicts. |
Integrated, thanks Will! |
...er for
Cluster Listeners.
https://issues.jboss.org/browse/ISPN-4753