dshunter107 commented on PR #2207:
URL: https://github.com/apache/streampipes/pull/2207#issuecomment-1832846974

   Take your time, No worries. 
   
   Sure, whatever works.  I included the timestamp as a long with milliseconds 
in the event as well as the time zone that the user specified. 
   I have a few clarifying questions: 
   
   (1)  Did you mean to say the "Zoned Date Time" object?
   (2) Are you saying that I should delete the "Zoned Date Time" object for 
now? 
   (3) If we have only the milliseconds, how would you like to handle different 
time zones? As of now, with only the milliseconds, if the data was taken in 
Florida at 4PM, a Streampipes user in California would have a reading for 1PM. 
This would result in the same instant, but a different time. Is that okay? I 
put the time zone provided by the user in the event as well, if we want to be 
able to specify the same instant but the time zone provided by the user. 
   (4) Could this issue be related to the OutputStrategy that I selected: 
   
   `.outputStrategy(OutputStrategies.append(
               EpProperties.timestampProperty(OUTPUT_DATETIME_RUNTIME_NAME)))`
   
   I ask because I am not sure if the timestampProperty was designed for 
ZonedDateTime. 
   
   


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