[jira] [Commented] (SPARK-10576) Move .java files out of src/main/scala
[ https://issues.apache.org/jira/browse/SPARK-10576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14742375#comment-14742375 ] Sean Owen commented on SPARK-10576: --- Yes a lot of these seem to be duplicating the {{package.scala}} file next to it in order to get a package summary to appear for these packages in the _Javadoc_ as well as Scaladoc. That seems legitimate and it's most of them. What about the others thought? the actual class files seem like they can go in {{src/main/java}}. Let me try moving them to see if it breaks, as a first pass. > Move .java files out of src/main/scala > -- > > Key: SPARK-10576 > URL: https://issues.apache.org/jira/browse/SPARK-10576 > Project: Spark > Issue Type: Improvement > Components: Build >Affects Versions: 1.5.0 >Reporter: Sean Owen >Priority: Minor > > (I suppose I'm really asking for an opinion on this, rather than asserting it > must be done, but seems worthwhile. CC [~rxin] and [~pwendell]) > As pointed out on the mailing list, there are some Java files in the Scala > source tree: > {code} > ./bagel/src/main/scala/org/apache/spark/bagel/package-info.java > ./core/src/main/scala/org/apache/spark/annotation/AlphaComponent.java > ./core/src/main/scala/org/apache/spark/annotation/DeveloperApi.java > ./core/src/main/scala/org/apache/spark/annotation/Experimental.java > ./core/src/main/scala/org/apache/spark/annotation/package-info.java > ./core/src/main/scala/org/apache/spark/annotation/Private.java > ./core/src/main/scala/org/apache/spark/api/java/package-info.java > ./core/src/main/scala/org/apache/spark/broadcast/package-info.java > ./core/src/main/scala/org/apache/spark/executor/package-info.java > ./core/src/main/scala/org/apache/spark/io/package-info.java > ./core/src/main/scala/org/apache/spark/rdd/package-info.java > ./core/src/main/scala/org/apache/spark/scheduler/package-info.java > ./core/src/main/scala/org/apache/spark/serializer/package-info.java > ./core/src/main/scala/org/apache/spark/util/package-info.java > ./core/src/main/scala/org/apache/spark/util/random/package-info.java > ./external/flume/src/main/scala/org/apache/spark/streaming/flume/package-info.java > ./external/kafka/src/main/scala/org/apache/spark/streaming/kafka/package-info.java > ./external/mqtt/src/main/scala/org/apache/spark/streaming/mqtt/package-info.java > ./external/twitter/src/main/scala/org/apache/spark/streaming/twitter/package-info.java > ./external/zeromq/src/main/scala/org/apache/spark/streaming/zeromq/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/impl/EdgeActiveness.java > ./graphx/src/main/scala/org/apache/spark/graphx/lib/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/TripletFields.java > ./graphx/src/main/scala/org/apache/spark/graphx/util/package-info.java > ./mllib/src/main/scala/org/apache/spark/ml/attribute/package-info.java > ./mllib/src/main/scala/org/apache/spark/ml/package-info.java > ./mllib/src/main/scala/org/apache/spark/mllib/package-info.java > ./sql/catalyst/src/main/scala/org/apache/spark/sql/types/SQLUserDefinedType.java > ./sql/hive/src/main/scala/org/apache/spark/sql/hive/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/api/java/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/dstream/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/StreamingContextState.java > {code} > It happens to work since the Scala compiler plugin is handling both. > On its face, they should be in the Java source tree. I'm trying to figure out > if there are good reasons they have to be in this less intuitive location. > I might try moving them just to see. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Commented] (SPARK-10576) Move .java files out of src/main/scala
[ https://issues.apache.org/jira/browse/SPARK-10576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14742653#comment-14742653 ] Apache Spark commented on SPARK-10576: -- User 'srowen' has created a pull request for this issue: https://github.com/apache/spark/pull/8736 > Move .java files out of src/main/scala > -- > > Key: SPARK-10576 > URL: https://issues.apache.org/jira/browse/SPARK-10576 > Project: Spark > Issue Type: Improvement > Components: Build >Affects Versions: 1.5.0 >Reporter: Sean Owen >Priority: Minor > > (I suppose I'm really asking for an opinion on this, rather than asserting it > must be done, but seems worthwhile. CC [~rxin] and [~pwendell]) > As pointed out on the mailing list, there are some Java files in the Scala > source tree: > {code} > ./bagel/src/main/scala/org/apache/spark/bagel/package-info.java > ./core/src/main/scala/org/apache/spark/annotation/AlphaComponent.java > ./core/src/main/scala/org/apache/spark/annotation/DeveloperApi.java > ./core/src/main/scala/org/apache/spark/annotation/Experimental.java > ./core/src/main/scala/org/apache/spark/annotation/package-info.java > ./core/src/main/scala/org/apache/spark/annotation/Private.java > ./core/src/main/scala/org/apache/spark/api/java/package-info.java > ./core/src/main/scala/org/apache/spark/broadcast/package-info.java > ./core/src/main/scala/org/apache/spark/executor/package-info.java > ./core/src/main/scala/org/apache/spark/io/package-info.java > ./core/src/main/scala/org/apache/spark/rdd/package-info.java > ./core/src/main/scala/org/apache/spark/scheduler/package-info.java > ./core/src/main/scala/org/apache/spark/serializer/package-info.java > ./core/src/main/scala/org/apache/spark/util/package-info.java > ./core/src/main/scala/org/apache/spark/util/random/package-info.java > ./external/flume/src/main/scala/org/apache/spark/streaming/flume/package-info.java > ./external/kafka/src/main/scala/org/apache/spark/streaming/kafka/package-info.java > ./external/mqtt/src/main/scala/org/apache/spark/streaming/mqtt/package-info.java > ./external/twitter/src/main/scala/org/apache/spark/streaming/twitter/package-info.java > ./external/zeromq/src/main/scala/org/apache/spark/streaming/zeromq/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/impl/EdgeActiveness.java > ./graphx/src/main/scala/org/apache/spark/graphx/lib/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/TripletFields.java > ./graphx/src/main/scala/org/apache/spark/graphx/util/package-info.java > ./mllib/src/main/scala/org/apache/spark/ml/attribute/package-info.java > ./mllib/src/main/scala/org/apache/spark/ml/package-info.java > ./mllib/src/main/scala/org/apache/spark/mllib/package-info.java > ./sql/catalyst/src/main/scala/org/apache/spark/sql/types/SQLUserDefinedType.java > ./sql/hive/src/main/scala/org/apache/spark/sql/hive/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/api/java/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/dstream/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/StreamingContextState.java > {code} > It happens to work since the Scala compiler plugin is handling both. > On its face, they should be in the Java source tree. I'm trying to figure out > if there are good reasons they have to be in this less intuitive location. > I might try moving them just to see. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Commented] (SPARK-10576) Move .java files out of src/main/scala
[ https://issues.apache.org/jira/browse/SPARK-10576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14742280#comment-14742280 ] Patrick Wendell commented on SPARK-10576: - FWIW - seems to me like moving them into /java makes sense. If we are going to have src/main/scala and src/main/java, might as well use them correctly. What do you think [~rxin]. > Move .java files out of src/main/scala > -- > > Key: SPARK-10576 > URL: https://issues.apache.org/jira/browse/SPARK-10576 > Project: Spark > Issue Type: Improvement > Components: Build >Affects Versions: 1.5.0 >Reporter: Sean Owen >Priority: Minor > > (I suppose I'm really asking for an opinion on this, rather than asserting it > must be done, but seems worthwhile. CC [~rxin] and [~pwendell]) > As pointed out on the mailing list, there are some Java files in the Scala > source tree: > {code} > ./bagel/src/main/scala/org/apache/spark/bagel/package-info.java > ./core/src/main/scala/org/apache/spark/annotation/AlphaComponent.java > ./core/src/main/scala/org/apache/spark/annotation/DeveloperApi.java > ./core/src/main/scala/org/apache/spark/annotation/Experimental.java > ./core/src/main/scala/org/apache/spark/annotation/package-info.java > ./core/src/main/scala/org/apache/spark/annotation/Private.java > ./core/src/main/scala/org/apache/spark/api/java/package-info.java > ./core/src/main/scala/org/apache/spark/broadcast/package-info.java > ./core/src/main/scala/org/apache/spark/executor/package-info.java > ./core/src/main/scala/org/apache/spark/io/package-info.java > ./core/src/main/scala/org/apache/spark/rdd/package-info.java > ./core/src/main/scala/org/apache/spark/scheduler/package-info.java > ./core/src/main/scala/org/apache/spark/serializer/package-info.java > ./core/src/main/scala/org/apache/spark/util/package-info.java > ./core/src/main/scala/org/apache/spark/util/random/package-info.java > ./external/flume/src/main/scala/org/apache/spark/streaming/flume/package-info.java > ./external/kafka/src/main/scala/org/apache/spark/streaming/kafka/package-info.java > ./external/mqtt/src/main/scala/org/apache/spark/streaming/mqtt/package-info.java > ./external/twitter/src/main/scala/org/apache/spark/streaming/twitter/package-info.java > ./external/zeromq/src/main/scala/org/apache/spark/streaming/zeromq/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/impl/EdgeActiveness.java > ./graphx/src/main/scala/org/apache/spark/graphx/lib/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/TripletFields.java > ./graphx/src/main/scala/org/apache/spark/graphx/util/package-info.java > ./mllib/src/main/scala/org/apache/spark/ml/attribute/package-info.java > ./mllib/src/main/scala/org/apache/spark/ml/package-info.java > ./mllib/src/main/scala/org/apache/spark/mllib/package-info.java > ./sql/catalyst/src/main/scala/org/apache/spark/sql/types/SQLUserDefinedType.java > ./sql/hive/src/main/scala/org/apache/spark/sql/hive/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/api/java/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/dstream/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/StreamingContextState.java > {code} > It happens to work since the Scala compiler plugin is handling both. > On its face, they should be in the Java source tree. I'm trying to figure out > if there are good reasons they have to be in this less intuitive location. > I might try moving them just to see. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Commented] (SPARK-10576) Move .java files out of src/main/scala
[ https://issues.apache.org/jira/browse/SPARK-10576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14742066#comment-14742066 ] Kiran Lonikar commented on SPARK-10576: --- thats right, the intent is to find out if there is any particular reason to colocate the java and scala files. Additionally, there are instances of scala files in java source tree. Did not make an attempt to find all, but here is one: core/src/main/java/org/apache/spark/api/java/function/package.scala > Move .java files out of src/main/scala > -- > > Key: SPARK-10576 > URL: https://issues.apache.org/jira/browse/SPARK-10576 > Project: Spark > Issue Type: Improvement > Components: Build >Affects Versions: 1.5.0 >Reporter: Sean Owen >Priority: Minor > > (I suppose I'm really asking for an opinion on this, rather than asserting it > must be done, but seems worthwhile. CC [~rxin] and [~pwendell]) > As pointed out on the mailing list, there are some Java files in the Scala > source tree: > {code} > ./bagel/src/main/scala/org/apache/spark/bagel/package-info.java > ./core/src/main/scala/org/apache/spark/annotation/AlphaComponent.java > ./core/src/main/scala/org/apache/spark/annotation/DeveloperApi.java > ./core/src/main/scala/org/apache/spark/annotation/Experimental.java > ./core/src/main/scala/org/apache/spark/annotation/package-info.java > ./core/src/main/scala/org/apache/spark/annotation/Private.java > ./core/src/main/scala/org/apache/spark/api/java/package-info.java > ./core/src/main/scala/org/apache/spark/broadcast/package-info.java > ./core/src/main/scala/org/apache/spark/executor/package-info.java > ./core/src/main/scala/org/apache/spark/io/package-info.java > ./core/src/main/scala/org/apache/spark/rdd/package-info.java > ./core/src/main/scala/org/apache/spark/scheduler/package-info.java > ./core/src/main/scala/org/apache/spark/serializer/package-info.java > ./core/src/main/scala/org/apache/spark/util/package-info.java > ./core/src/main/scala/org/apache/spark/util/random/package-info.java > ./external/flume/src/main/scala/org/apache/spark/streaming/flume/package-info.java > ./external/kafka/src/main/scala/org/apache/spark/streaming/kafka/package-info.java > ./external/mqtt/src/main/scala/org/apache/spark/streaming/mqtt/package-info.java > ./external/twitter/src/main/scala/org/apache/spark/streaming/twitter/package-info.java > ./external/zeromq/src/main/scala/org/apache/spark/streaming/zeromq/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/impl/EdgeActiveness.java > ./graphx/src/main/scala/org/apache/spark/graphx/lib/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/package-info.java > ./graphx/src/main/scala/org/apache/spark/graphx/TripletFields.java > ./graphx/src/main/scala/org/apache/spark/graphx/util/package-info.java > ./mllib/src/main/scala/org/apache/spark/ml/attribute/package-info.java > ./mllib/src/main/scala/org/apache/spark/ml/package-info.java > ./mllib/src/main/scala/org/apache/spark/mllib/package-info.java > ./sql/catalyst/src/main/scala/org/apache/spark/sql/types/SQLUserDefinedType.java > ./sql/hive/src/main/scala/org/apache/spark/sql/hive/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/api/java/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/dstream/package-info.java > ./streaming/src/main/scala/org/apache/spark/streaming/StreamingContextState.java > {code} > It happens to work since the Scala compiler plugin is handling both. > On its face, they should be in the Java source tree. I'm trying to figure out > if there are good reasons they have to be in this less intuitive location. > I might try moving them just to see. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org