go-jmespath icon indicating copy to clipboard operation
go-jmespath copied to clipboard

Audit existing jmespath functions for reflection usage

Open jamesls opened this issue 9 years ago • 2 comments

We need to avoid checks for empty interfaces and map[string]interface{} and instead use reflection. This has been fixed in interpreter.go but functions.go needs the same treatment.

I'd also like to investigate adding autogen'd tests using structs from the compliance tests to verify we don't miss anything.

Related: https://github.com/jmespath/go-jmespath/issues/14

jamesls avatar Feb 02 '16 18:02 jamesls

I'm new to this jmespath implementation, but I was hoping that it didn't use reflection given the rather steep performance penalty. Are you basically moving away from type assertions to reflection across the board or only in very narrow cases?

tve avatar May 18 '16 21:05 tve

This is 4 years old. What's going on here?

skyzyx avatar Apr 01 '20 20:04 skyzyx