WingsGo commented on pull request #4009: URL: https://github.com/apache/incubator-doris/pull/4009#issuecomment-654643098
> > > It's better add more discrete message in commit info like you issue. > > > > > > Ok, done, and I tested the schema change supported in doc, but I found that convert LargetInt to Double is not support in code implement, maybe you can help me confirm that why it is added to doc in case It is supported but there is some bug exists? > > I test the case, it's forbidden by FE to change LARGEINT to DOUBLE. Because LARGEINT is int128_t and double is 2^52 and will lose information. > So you can remove from the doc. Ok, I removed it, thx~ ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
