Re: [DISCUSS] Apache Polaris powered by Quarkus

2024-11-19 Thread Michael Collado
Hey thanks for calling that out. It’s strictly a convenience change and actually isn’t necessary. I added that in because, at the end of the day, most components need a PolarisMetaStoreManager, not a MetaStoreManagerFactory. Registering it as a factory meant that components could declare a dependen

Re: [DISCUSS] Apache Polaris powered by Quarkus

2024-11-19 Thread Dmitri Bourlatchkov
I had a quite brief look at Mike's HK2 branch, so apologies if I'm missing the bigger picture. Still, I see that MetaStoreManagerFactory no longer extends the DW Discoverable, but it extends the HK2 Factory class now. So we're basically trading one framework for another as a code-level runtime dep