Re: LayerLengthEv and NoCacheEvaluatable - Layer API issue

2023-09-14 Thread Mike Beckerle
On Thu, Sep 14, 2023 at 1:06 PM Steve Lawrence wrote: > ... > > And with changes to the API, we could maybe even avoid #1 without > ThreadLocals. For example, instead of using a LayerTransformerFactory to > create new LayerTransformer instances, maybe we just directly create new > instances of

Re: LayerLengthEv and NoCacheEvaluatable - Layer API issue

2023-09-14 Thread Steve Lawrence
All fair points. After becoming a bit more familiar with how layers work, I have some additional thoughts. Currently the way layers work is we create a single LayerTransformerFactory during schema compilation, and each time parse/unparse comes across this layer it calls factory.newInstance()