rusackas commented on PR #40918: URL: https://github.com/apache/superset/pull/40918#issuecomment-5094182745
Appreciate the thorough writeup and the real Impala validation. Since this is explicitly a PoC for design feedback and not for merge, I'm not blocking on anything. That said, we'll want the conflict cleared so reviewers can actually check it out and click through the UI changes. The scoping (off-by-default flag, Postgres+Impala only, no per-user zone) all seems sane to me. Will hold off on a deeper design opinion until it's rebased and I can run it locally. -- 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]
