JavaLionLi commented on PR #637: URL: https://github.com/apache/dubbo-spi-extensions/pull/637#issuecomment-2982373749
> if we accept this pr, i doubt the simplest way might be deleting jedisParams from RedisMetadataReport.java because all metadata might not be cleaned after 48 hours. @heliang666s If you delete this, it will be meaningless, and the first time I implemented the class with nacos was because the metadata would not automatically expire, resulting in metadata inflation And this is controlled by the TTL of Redis, and it can be deleted after 48 hours The main problem with this bug is that he deletes something that shouldn't be deleted, causing the service to not find metadata and get an error There is no problem with the 48-hour expiration of this one -- 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: notifications-unsubscr...@dubbo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.org For additional commands, e-mail: notifications-h...@dubbo.apache.org
