reuvenlax commented on issue #25228: URL: https://github.com/apache/beam/issues/25228#issuecomment-1420116876
This is an interesting request. Even in the above example, not all INT64 values can be converted to Java double. e.g. the following code: long longValue = XXXX; double doubleValue = (double) longValue; assert(longValue == (long) doubleValue); will often fail, especially for large numbers (e.g. try it out for 499999999000000001L). I guess we could convert optimistically and fail if the specific number happens to lose precision, however I'm uncomfortable with that. That's a recipe for a pipeline that works for a while and then suddenly starts behaving poorly as the input data changes, which is very confusing. I think understandable and easily-predictable behavior is better. -- 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]
