mattcasters commented on PR #8270: URL: https://github.com/apache/hop/pull/8270#issuecomment-5559445847
Follow-up pass on the review fixes: the previous findings are addressed. One leftover in `NotificationFetchCache`: it was keyed only by source id, so in Hop Web a session that had not reloaded after an edit could refill the process-wide cache and hand the old feed to the session that just saved. Pushed `e553f56c4d` onto this branch — the key is now source id plus the provider description (RSS URL / GitHub repo), with tests for that case. -- 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]
