Add filter command as an alias for map(select())#3180
Add filter command as an alias for map(select())#3180bseidenberg wants to merge 1 commit intojqlang:masterfrom
filter command as an alias for map(select())#3180Conversation
I (and I've heard others) use this often enough that it makes sense to have more user-friendly syntax.
|
Not sure what is needed to get this PR reviewed - please let me know. |
|
Hi, the code itself looks ok but don't think i've heard anyone ask for such addition before. Maybe other maintainers have? we're usually quite conservative with adding new functions. |
|
@wader wrote:
Not so long ago, there was a major performance issue that justified such conservatism. It's my understanding that the performance overhead problem has been largely resolved. Does anyone know what the performance overhead these days is? Even setting aside general considerations concerning bloat, the argument against adding Another, though admittedly lesser, point concerns the wisdom of having two ways to say (in words) the same thing. ("In words" because I do not mean to include symbolic expressions like |
Don't know but it's my understand also that performance of builtins.jq is not a big issue now. I was mostly thinking about the same issues you also pointed out, adding on to already confusing names and partially overlapping functionality. |
I (and I've heard others) use this often enough that it makes sense to have more user-friendly syntax.