Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/1213
I just pushed a commit out based on some feedback from Nick:
- The stellar context is now passed into the ParserRunner abstraction as a
dependency. I imagine different environments will require different Stellar
initialization strategies so this made sense to me. I also like it because it
makes the abstraction simpler.
- Changed ParserRunner to an interface with a single implementation. I
could change this to ParserRunnerStrategy in case we think we might want to use
that pattern in the future. Then all that would be missing is a ParserContext
class (and possibly a ParserRunnerStrategies enum) which we could add now or
when we need it.
---
