[GitHub] [spark] cloud-fan edited a comment on pull request #28692: [SPARK-31879][SQL] Using GB as default Locale for datetime formatters

[GitBox <gi...@apache.org>]

cloud-fan edited a comment on pull request #28692:
URL: https://github.com/apache/spark/pull/28692#issuecomment-638147190


   > please also create a PR to explain what gets changed in the meaning of u in the migration guide.
   
   @bart-samwel maybe you were confused by this. The meaning was already changed when we switch to the java 8 formatter API. In the legacy formatter, `u` means `Day number of week (1 = Monday, ..., 7 = Sunday)`, which is number, in the new formatter, we replace `u` with `e`, but `e` is `localized day-of-week`, which is number/text.
   
   There is no such a pattern letter in the java 8 formatter API, which has the same meaning of the legacy `u`, the `e` is the closest.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org