[GitHub] [spark] srowen commented on issue #25186: [SPARK-28433][SQL]Special scala test case for aarch64

2019-07-18 Thread GitBox
srowen commented on issue #25186: [SPARK-28433][SQL]Special scala test case for aarch64 URL: https://github.com/apache/spark/pull/25186#issuecomment-513090568 (Yes, this assertion fails on ARM, per the thread on dev@) This is

[GitHub] [spark] srowen commented on issue #25186: [SPARK-28433][SQL]Special scala test case for aarch64

2019-07-18 Thread GitBox
srowen commented on issue #25186: [SPARK-28433][SQL]Special scala test case for aarch64 URL: https://github.com/apache/spark/pull/25186#issuecomment-512815515 I think this change is narrowly fine. I'm actually not sure why we test the result of this JDK method directly; it's not a Spark me