adamdebreceni commented on a change in pull request #976:
URL: https://github.com/apache/nifi-minifi-cpp/pull/976#discussion_r559356325
##########
File path: extensions/windows-event-log/ConsumeWindowsEventLog.h
##########
@@ -43,9 +43,14 @@ namespace minifi {
namespace processors {
struct EventRender {
- std::map<std::string, std::string> matched_fields_;
- std::string text_;
- std::string rendered_text_;
+ std::map<std::string, std::string> matched_fields;
+ std::string xml;
+ std::string plaintext;
+ struct {
+ std::string raw;
+ std::string simple;
+ std::string flattened;
+ } json;
Review comment:
based on the recently reopened #618 in the future we will be able to
specify multiple values for various properties, e.g. in this case `Output
Format` will be able to take a list of formats, (there is already some hack
with `Both` for this purpose), so the current solution takes that direction
into consideration
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
