muyangye commented on issue #2106:
URL: https://github.com/apache/streampipes/issues/2106#issuecomment-1811380979

   > My suggestion was to work directly on option 2. The idea behind this was 
that you then don't have to make any notifications to the user, as arrays are 
processed by the DataLake sink as expected. I thought this should be easier to 
implement, as it would "only" mean changes in the `InfluxStore` class. More 
precisely, only a condition for `EventPropertyList` would have to be added in 
the `onEvent` method.
   > 
   > I thought that it would be easier to implement then an exception handling 
for array properties. Do you see it the same way? This was only meant as a 
suggestion, if you think it's easier otherwise that's totally ok for me .
   
   I see, I just realized after serializing the array to string the influx 
store will store it which will be displayed in the dashboard (as opposed to 
nothing) so the user will notice a difference between the current situation. 
Yes I think it is definitely better. Thanks a lot for your suggestion!


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to