Evan Rusackas
Evan Rusackas
Merged! Thanks again for the help with this!
It looks like CI is stuck because of some tests failing (which I'm re-running), and pre-commit checks (i.e. linting) so you will need to run the Pre-commit hooks Superset uses...
I think the main thing is that we can't impact performance. @dpgaspar is currently out of office, but may be able to provide suggestions upon his return of how to...
This seems like it ties in to other [proposals](https://github.com/apache/superset/issues/27194) and [efforts](https://github.com/apache/superset/pull/30398) involving using UUIDs more systemically, so I'll ping @hondyman and @mistercrunch to take a look here too, as well...
Oh, and this will require a revisit from @michael-s-molina to un-block, if his performance concerns are indeed resolved :)
Sorry, this seems to have slipped under the collective radar. Could you give it a rebase and we'll try to take a fresh look?
Just FYI, I set this to auto-close https://github.com/apache/superset/issues/29879, since... I think it does!
Still hoping to get this one through if we can... thanks for your patience :D Looks like it needs a rebase... not sure what's up with the other test failures,...
Looks like this could use a rebase, which might resolve some CI failures if we're lucky. Only @michael-s-molina can remove the block, though.
Looks like this could use a rebase, which might resolve some CI failures if we're lucky. Only @michael-s-molina can remove the block, though.