Christopher J. Morrone

> At this point, I think we should revert this change and replace it with the extra option(s) for the load command. I don't think this has landed yet. Nothing...

Yes, there was probably a warning the "configure" output, but AC_PROG_LEX doesn't error out when a lexer isn't found. We should probably add an autoconf test that checks if it...

The combined allocate+append sounds reasonable to me. Also, speaking of ldms_list_remove_item(), it might be good to expand the comment to mention records or something. It was not clear to me...

> @narategithub, @nichamon, @morrone I am inclined to keep the API as designed/written and update the documentation to make it clear exactly what is being done so that it is...

> @morrone, @narategithub, @nichamon we implemented arrays of records only because we could. I think we should consider whether or not this is wise. Arrays are fixed size and this...

Agreed! We needed to add an elasticsearch ingest pipeline to strip the id, but for other reasons. Not ideal. Giving us a a connector configuration option to opt out of...

Yes, making it configurable would be great. We would want to configure the maximum to be something like 5-10 minutes.

> Please verify if this is how you imagined it. If its good I can ask the maintainers to have a look. Yes, that looks perfect. Thank you!

Point me to where that was documented in the code?

I don't remember these being properly documented.