Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/7375#discussion_r34497201
  
    --- Diff: core/src/test/java/org/apache/spark/JavaAPISuite.java ---
    @@ -63,6 +63,7 @@
     // The test suite itself is Serializable so that anonymous Function 
implementations can be
     // serialized, as an alternative to converting these anonymous classes to 
static inner classes;
     // see http://stackoverflow.com/questions/758570/.
    +@SuppressWarnings("serial")
    --- End diff --
    
    The warning is bogus though, about serialVersionUID? it should actually not 
be specified by default. Does the Spark build actually enable this warning 
though, or is it just Eclipse? I didn't think -Xlint was on.
    
    This can't be the only warning :) I see a load still, that I've been 
meaning to fix again again.


---
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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to