[GitHub] [spark] srowen commented on issue #25133: [SPARK-28365][ML] Fallback locale to en_US in StopWordsRemover if system default locale isn't in available locales in JVM

2019-07-18 Thread GitBox
srowen commented on issue #25133: [SPARK-28365][ML] Fallback locale to en_US in 
StopWordsRemover if system default locale isn't in available locales in JVM
URL: https://github.com/apache/spark/pull/25133#issuecomment-512835449
 
 
   I am not sure we should make this change if this is already resolvable by:
   - Fixing the underlying problem, which is really the OS locale
   - Specifying the en-US locale directly in StopWordsRemover


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


With regards,
Apache Git Services

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



[GitHub] [spark] srowen commented on issue #25133: [SPARK-28365][ML] Fallback locale to en_US in StopWordsRemover if system default locale isn't in available locales in JVM

2019-07-16 Thread GitBox
srowen commented on issue #25133: [SPARK-28365][ML] Fallback locale to en_US in 
StopWordsRemover if system default locale isn't in available locales in JVM
URL: https://github.com/apache/spark/pull/25133#issuecomment-511785224
 
 
   Yeah, I'm just wondering - wouldn't this cause a hundred other problems with 
JVM-based apps?


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


With regards,
Apache Git Services

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



[GitHub] [spark] srowen commented on issue #25133: [SPARK-28365][ML] Fallback locale to en_US in StopWordsRemover if system default locale isn't in available locales in JVM

2019-07-15 Thread GitBox
srowen commented on issue #25133: [SPARK-28365][ML] Fallback locale to en_US in 
StopWordsRemover if system default locale isn't in available locales in JVM
URL: https://github.com/apache/spark/pull/25133#issuecomment-511649524
 
 
   I see, that seems to be the root of the problem. And the OS doesn't include 
any localization for the JVM to en-TW?
   
   I mean, won't there be a lot of problems if the JVM can't access the default 
locale? I'm just wondering if it's worth patching this one with a potentially 
problematic behavior change in this situation vs throwing an error.
   
   Can you just force the host OS default locale in your shell to en?


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


With regards,
Apache Git Services

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



[GitHub] [spark] srowen commented on issue #25133: [SPARK-28365][ML] Fallback locale to en_US in StopWordsRemover if system default locale isn't in available locales in JVM

2019-07-15 Thread GitBox
srowen commented on issue #25133: [SPARK-28365][ML] Fallback locale to en_US in 
StopWordsRemover if system default locale isn't in available locales in JVM
URL: https://github.com/apache/spark/pull/25133#issuecomment-511424823
 
 
   Yeah but how does en-TW become the default if it isn't supported? Isn't this 
the problem?


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


With regards,
Apache Git Services

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