You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have a use case in search-a-licious where I wanted to add some parameter to EAbstractNode.
Note also, that as those parameters are query dependent and creating an ElasticsearchQueryBuilder instance is a bit time consuming, I want to do this at query generation time and not at class instantiation time.
It's currently difficult to do that without re-implementing part of the methods of the visitor.
It would be easier if:
we enable adding an initial context in __call__ and visit methods
we pass along this context each time we build an EAbstractNode
That way specific classes of EAbstractNode can tweak their result as much as they want with a minimal changes to ElasticsearchQueryBuilder.
The text was updated successfully, but these errors were encountered:
I have a use case in search-a-licious where I wanted to add some parameter to EAbstractNode.
Note also, that as those parameters are query dependent and creating an ElasticsearchQueryBuilder instance is a bit time consuming, I want to do this at query generation time and not at class instantiation time.
It's currently difficult to do that without re-implementing part of the methods of the visitor.
It would be easier if:
__call__
andvisit
methodsThat way specific classes of EAbstractNode can tweak their result as much as they want with a minimal changes to ElasticsearchQueryBuilder.
The text was updated successfully, but these errors were encountered: