suddjian commented on pull request #16077: URL: https://github.com/apache/superset/pull/16077#issuecomment-894447047
@zhaoyongjie I'm not sure exactly where it happens either, but one of @betodealmeida's changes started saving `query_context`. It's not unheard of for `GET` requests to involve db writes, when the server wants to save some aspect of a request or lazily calculate and then save a value, so I think it's okay. The main thing here is that the value being used for "last modified" in the UI should not be something that auto-updates when a database row is written. This PR makes it more explicit. -- 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]
