[GitHub] spark issue #14232: [SPARK-16587] [CORE] The annotation for the abstract Cla...

2016-08-04 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/14232 @f7753 I think just listing the message types here is not super useful. Yes, new messages were added and not mentioned in the documentation, but then adding just "foo is used by bar" doesn't really

[GitHub] spark issue #14232: [SPARK-16587] [CORE] The annotation for the abstract Cla...

2016-07-29 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14232 Hm, not sure what to do here. Unless we can get an informed double-check on the correctness of this, I think I'd just not do it. --- If your project is set up for it, you can reply to this email

[GitHub] spark issue #14232: [SPARK-16587] [CORE] The annotation for the abstract Cla...

2016-07-24 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14232 I'm not clear if this matches the intent of the other docs. I'd close this unless we can get a clearer picture of the intent of these messages. --- If your project is set up for it, you can reply

[GitHub] spark issue #14232: [SPARK-16587] [CORE] The annotation for the abstract Cla...

2016-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14232 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this