Abhishek-kumar-samsung commented on PR #24449: URL: https://github.com/apache/superset/pull/24449#issuecomment-2011341422
> I agree we don't want to grow the scope of this PR. We should merge this first, and then deal with the cron idea (which is a nice idea) as a subsequent PR. > > For the cron feature, it seems like a reasonable plan to seek "lazy consensus" on the dev@ list. Then if that starts an argument, we can upgrade the conversation to a SIP and vote it. Yes sure, i will quickly do the two changes that you had suggested and once merged, we will raise SIP for cron. Then we can work on it, @coetzeevs you can also work on that if you had implemented something in 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]
