enhancementhelp wanted
Métricas do repositório
- Stars
- (11.537 estrelas)
- Métricas de merge de PR
- (Mesclagem média 5d 18h) (41 fundiu PRs em 30d)
Description
In #1058, we did one step from (dynamic factory + json) to a static enum world in order to have source type specific source params validation.
We are now half-way between two worlds.
Solution 1
Just go full enum. The factory is a function of the enum. Pros: Very simple Cons: Does not separate the concern super well.
Note: If we go for that route, source_type() should return an enum too.
Solution 2
Use a SourceParams trait that can build a Source type (does the job of the factory) and do dark magic, like what @fmassot did for the IndexConfig deserialization.