datawave icon indicating copy to clipboard operation
datawave copied to clipboard

Create JexlNodeBuilder for creating JexlNodes

Open lbschanno opened this issue 3 years ago • 2 comments

Create an initial builder library for creating JexlNodes. This library is aimed at reducing the verbosity involved in creating new JexlNode instances, and at reducing bugs introduced by either incorrectly setting or forgetting to set the correct parentage for child nodes.

This pull request also contains a proof-of-concept of the builder library being put to use in JexlNodeFactory for several methods.

lbschanno avatar Dec 28 '21 08:12 lbschanno

nice work!

ivakegg avatar Jan 26 '22 14:01 ivakegg

@jwomeara I haven't done any extensive performance testing, but you make a good point of its importance, I'll try to get some performance metrics for you.

lbschanno avatar Apr 25 '22 23:04 lbschanno