dpol1 commented on PR #3011:
URL: https://github.com/apache/hugegraph/pull/3011#issuecomment-4334740756

   Hi @imbajin - while working on this PR I spotted what might be a separate 
bug in the legacy `EventHub` path. Flagging before I file anything.
   
   Producers emit past-tense actions:
   - `Cache.ACTION_INVALIDED` (`"invalided"`)
   - `Cache.ACTION_CLEARED` (`"cleared"`)
   
   Listeners only match the present-tense ones:
   - `Cache.ACTION_INVALID` (`"invalid"`)
   - `Cache.ACTION_CLEAR` (`"clear"`)
   
   So those events look like they get dropped silently. Wider blast radius than 
the HStore `CachedSchemaTransactionV2` fix here, so I kept it out of scope.
   
   Worth a separate issue - or is this intentional? If you're +1, I'll take 
care of it
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to