Skip to content

moved query tranformer registry to the worker

Administrator requested to merge twop/migrate-query-transformer into master

Created by: twop

This is early WIP for major reworking of query transformers:

done:

  • removed query transformer registry
  • moved tracking of transformers to the worker thread
  • made chain of transformation more explicit
  • added optimization to avoid going to the worker thread if there are no transformers registered
  • removed Client and Ext Query transformers classes

left

  • rework/removed existing tests
  • add new ones for edge cases (such as unsubbing in the middle of transformation)

I believe that avoiding going to the worker thread is needed because query transformers are rare, thus it makes sense not to pay cost of the thread-hop in majority of cases.

Looking for early feedback if this approach is viable.

Merge request reports