[jira] [Commented] (SPARK-7987) TransportContext.createServer(int port) is missing in Spark 1.4
[ https://issues.apache.org/jira/browse/SPARK-7987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14567556#comment-14567556 ] Marcelo Vanzin commented on SPARK-7987: --- [~joshrosen] that annotation is nice but it cannot live in {{core/}} if this module is to use it. Actually, it would be really nice to have a new top-level module for these annotations and other very generic helper code (such as JavaUtils.java, which is used in more than the network module). TransportContext.createServer(int port) is missing in Spark 1.4 --- Key: SPARK-7987 URL: https://issues.apache.org/jira/browse/SPARK-7987 Project: Spark Issue Type: Bug Components: Shuffle Affects Versions: 1.4.0 Reporter: Patrick Wendell Priority: Blocker From what I can tell the SPARK-6229 patch removed this API: https://github.com/apache/spark/commit/38d4e9e446b425ca6a8fe8d8080f387b08683842#diff-d9d4b8d8e82b7d96d5e779353e4b2f4eL85 I think adding it back should be easy enough, but I cannot figure out why this didn't trigger MIMA errors. I am wondering if MIMA was not enabled properly for some of the new modules: /cc [~vanzin] [~rxin] and [~adav] I put this as a blocker level issue because I'm wondering if we just aren't enforcing checks for some reason in some of our API's. So I think we need to block the 1.4 release on at least making sure no other serious API's were broken. If it turns out only this API was affected, or I'm just missing something, we can downgrade it. -- 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-7987) TransportContext.createServer(int port) is missing in Spark 1.4
[ https://issues.apache.org/jira/browse/SPARK-7987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14566862#comment-14566862 ] Josh Rosen commented on SPARK-7987: --- [~pwend...@gmail.com], we now have the {{@Private}} annotation for this exact purpose (annotating Java classes which need to be public due to how they're used internally in Spark). TransportContext.createServer(int port) is missing in Spark 1.4 --- Key: SPARK-7987 URL: https://issues.apache.org/jira/browse/SPARK-7987 Project: Spark Issue Type: Bug Components: Shuffle Affects Versions: 1.4.0 Reporter: Patrick Wendell Priority: Blocker From what I can tell the SPARK-6229 patch removed this API: https://github.com/apache/spark/commit/38d4e9e446b425ca6a8fe8d8080f387b08683842#diff-d9d4b8d8e82b7d96d5e779353e4b2f4eL85 I think adding it back should be easy enough, but I cannot figure out why this didn't trigger MIMA errors. I am wondering if MIMA was not enabled properly for some of the new modules: /cc [~vanzin] [~rxin] and [~adav] I put this as a blocker level issue because I'm wondering if we just aren't enforcing checks for some reason in some of our API's. So I think we need to block the 1.4 release on at least making sure no other serious API's were broken. If it turns out only this API was affected, or I'm just missing something, we can downgrade it. -- 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-7987) TransportContext.createServer(int port) is missing in Spark 1.4
[ https://issues.apache.org/jira/browse/SPARK-7987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14566860#comment-14566860 ] Patrick Wendell commented on SPARK-7987: Ah I see - since this is all bytecode public should we annotate them or do something else to indicate clearly it's private? Or maybe it's presumed to be obvious? I have been wondering if we should have an explicit private annotation for Java classes we don't intend users to use. TransportContext.createServer(int port) is missing in Spark 1.4 --- Key: SPARK-7987 URL: https://issues.apache.org/jira/browse/SPARK-7987 Project: Spark Issue Type: Bug Components: Shuffle Affects Versions: 1.4.0 Reporter: Patrick Wendell Priority: Blocker From what I can tell the SPARK-6229 patch removed this API: https://github.com/apache/spark/commit/38d4e9e446b425ca6a8fe8d8080f387b08683842#diff-d9d4b8d8e82b7d96d5e779353e4b2f4eL85 I think adding it back should be easy enough, but I cannot figure out why this didn't trigger MIMA errors. I am wondering if MIMA was not enabled properly for some of the new modules: /cc [~vanzin] [~rxin] and [~adav] I put this as a blocker level issue because I'm wondering if we just aren't enforcing checks for some reason in some of our API's. So I think we need to block the 1.4 release on at least making sure no other serious API's were broken. If it turns out only this API was affected, or I'm just missing something, we can downgrade it. -- 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-7987) TransportContext.createServer(int port) is missing in Spark 1.4
[ https://issues.apache.org/jira/browse/SPARK-7987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14566857#comment-14566857 ] Reynold Xin commented on SPARK-7987: I don't think mima runs on the network module, which is not a public API. Why do we care about TransportContext.createServer? TransportContext.createServer(int port) is missing in Spark 1.4 --- Key: SPARK-7987 URL: https://issues.apache.org/jira/browse/SPARK-7987 Project: Spark Issue Type: Bug Components: Shuffle Affects Versions: 1.4.0 Reporter: Patrick Wendell Priority: Blocker From what I can tell the SPARK-6229 patch removed this API: https://github.com/apache/spark/commit/38d4e9e446b425ca6a8fe8d8080f387b08683842#diff-d9d4b8d8e82b7d96d5e779353e4b2f4eL85 I think adding it back should be easy enough, but I cannot figure out why this didn't trigger MIMA errors. I am wondering if MIMA was not enabled properly for some of the new modules: /cc [~vanzin] [~rxin] and [~adav] I put this as a blocker level issue because I'm wondering if we just aren't enforcing checks for some reason in some of our API's. So I think we need to block the 1.4 release on at least making sure no other serious API's were broken. If it turns out only this API was affected, or I'm just missing something, we can downgrade it. -- 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