[GitHub] [spark] HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper

HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper URL: https://github.com/apache/spark/pull/26397#issuecomment-552690252 Thanks all for reviewing and merging!

[GitHub] [spark] HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper

HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper URL: https://github.com/apache/spark/pull/26397#issuecomment-55281 Bump This is an

[GitHub] [spark] HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper

HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper URL: https://github.com/apache/spark/pull/26397#issuecomment-550132958 OK, I added the test which fails on master branch and passes with the patch. I guess

[GitHub] [spark] HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper

HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper URL: https://github.com/apache/spark/pull/26397#issuecomment-550101680 Thanks for the quick response! That's actually one of things what I tried - no luck.

[GitHub] [spark] HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper

HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper URL: https://github.com/apache/spark/pull/26397#issuecomment-550086482 Actually I have been trying to construct/modify UT to let it fail on master and pass

[GitHub] [spark] HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper

HeartSaVioR commented on issue #26397: [SPARK-29755][CORE] Provide @JsonDeserialize for Option[Long] in LogInfo & AttemptInfoWrapper URL: https://github.com/apache/spark/pull/26397#issuecomment-549739738 cc. @vanzin This is a follow-up patch of SPARK-28869. Thanks in advance.