lmccay
lmccay
> It looks like the project is being reinstated!! @gabrielmagno looks like there are some new commits including a fix for CI. > > While I don't think the team...
@the-other-tim-brown - I created a DRAFT pull request. This has gotten kind of large and still requires some additional testing and cleanup but is getting close. My testing is limited...
@the-other-tim-brown - that sounds interesting. Do you happen to have a write up about this? Would be a great resource for dev environments!
I like the idea of cleaning this up. I've been struggling what the "perTable" actually means in the name of this. It doesn't really have a per table cardinality within...
I'm picturing a flow where there is a SyncSubmit class that gets handed a config like the above which comes from a YAML file or programmatically assembled then uses the...
@ashvina - perhaps the injection should actually be done at the property level. Each client provides a setter for each param that it requires and the SyncSubmit code would pull...
@ashvina and @the-other-tim-brown - I've just been playing around with rolling our own injection mechanism to address PerTableConfig. I think that this will greatly simplify what we need to do....
@the-other-tim-brown - sure. Check out the PR #307 as a POC implementation. It works nicely and removes the need for all those other interfaces in the api module. I think...
@ashvina , @the-other-tim-brown - can we step back and revisit problem statement for the PerTableConfig refactoring here? I think that #293 related topics regarding format specific details leaking into various...