lhotari commented on PR #24842: URL: https://github.com/apache/pulsar/pull/24842#issuecomment-3416606971
> I do have two main concerns regarding potential edge cases, and I would appreciate your guidance on them. > > 1. If a mapper function returns null: What would be the most appropriate way for the TableView to handle this? In that case, the key is removed from the map since null is considered to be a tombstone value. > 2. If the mapper throws an exception: What is the desired behavior in this scenario? Good question. I don't think that this PIP impacts this. If the mapper throws an exception, the same message will get retried, so the table view will get stuck. The table view doesn't have any way to handle "poison pill" messages which fail every time. I think addressing this is out of context for this PIP. > Are these concerns that I should address in the implementation, or are they handled by other parts of the system and not something I need to consider? > > If my understanding of the direction is correct, I'd like to update the PIP document and the implementation PR accordingly. Please let me know if this is the right way to proceed. Makes sense, please go ahead. -- 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]
