obermeier commented on PR #1165: URL: https://github.com/apache/streampipes/pull/1165#issuecomment-1407781682
> Hello @obermeier, thanks for opening the issue and providing a PR. > > I recently encountered the same issue and changed it in [PR-1099](https://github.com/apache/streampipes/pull/1099/files). However, I suspect that the changes only affect the CSV format, as I tried your example and got the same behavior. > > The default should be that we always assume a floating point number. If the number is too large for a float (e.g. a Unix timestamp in ms) it is assumed to be a double (see class 'DatatypeUtils'). We decided to do this because if the first value of the CSV file is represented as an integer 1 and not as 1.0, when another value is then a float (e.g. 2.4) we will get runtime exceptions when storing the data in the time series storage. > > Your implementation now implements the same behavior, correct? Do your changes also affect the values at runtime or only the description of the event? > Hello @obermeier, thanks for opening the issue and providing a PR. > > I recently encountered the same issue and changed it in [PR-1099](https://github.com/apache/streampipes/pull/1099/files). However, I suspect that the changes only affect the CSV format, as I tried your example and got the same behavior. > > The default should be that we always assume a floating point number. If the number is too large for a float (e.g. a Unix timestamp in ms) it is assumed to be a double (see class 'DatatypeUtils'). We decided to do this because if the first value of the CSV file is represented as an integer 1 and not as 1.0, when another value is then a float (e.g. 2.4) we will get runtime exceptions when storing the data in the time series storage. > > Your implementation now implements the same behavior, correct? Do your changes also affect the values at runtime or only the description of the event? Hi @tenthe, I like the Idear to use just Floats and Doubles. A timestamp in ms shoud fit to this model. I ralized that I could not reproduce my "bug". -- 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]
