[jira] [Commented] (HAMA-925) OffHeapVerticesInfo should implement removeVertex
[ https://issues.apache.org/jira/browse/HAMA-925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14294447#comment-14294447 ] Hudson commented on HAMA-925: - SUCCESS: Integrated in Hama-Nightly-for-Hadoop-1.x #1504 (See [https://builds.apache.org/job/Hama-Nightly-for-Hadoop-1.x/1504/]) HAMA-925 - added OffHeapVerticesInfo#removeVertex and #finishRemovals implementations (tommaso: rev 1655029) * /hama/trunk/graph/src/main/java/org/apache/hama/graph/OffHeapVerticesInfo.java OffHeapVerticesInfo should implement removeVertex - Key: HAMA-925 URL: https://issues.apache.org/jira/browse/HAMA-925 Project: Hama Issue Type: Improvement Components: graph Affects Versions: 0.6.4 Reporter: Tommaso Teofili Assignee: Tommaso Teofili Fix For: 0.7.0 As brought up by [~udanax] and discussed on the ML OffHeapVerticesInfo#removeVertex should call CacheService#free to properly remove the passed vertex from DM. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
Re: Q, OffHeapVerticesInfo, CacheService.free() method for remove vertex
I've fixed it in HAMA-925,
Regards,
Tommaso
2015-01-27 8:50 GMT+01:00 Tommaso Teofili tommaso.teof...@gmail.com:
if I recall correctly the VerticesInfo#removeVertex API was not used at
the time I implemented the OffHeapVerticesInfo, that's why it was not
implemented, however if you refer to [1] I'd say that's surely a good idea
to implement it using that API.
Regards,
Tommaso
[1] :
http://directmemory.apache.org/apidocs/reference/org/apache/directmemory/cache/CacheService.html#free(K)
2015-01-27 6:00 GMT+01:00 Edward J. Yoon edward.y...@samsung.com:
Hi,
It seems CacheService.free() method can be used to remove vertex.
Is there a specific reason why you throw UnsupportedOperationException?
@Override
public void removeVertex(V vertexID) {
throw new UnsupportedOperationException(Not yet implemented);
}
--
Best Regards, Edward J. Yoon
RE: Q, OffHeapVerticesInfo, CacheService.free() method for remove vertex
Great job!
--
Best Regards, Edward J. Yoon
-Original Message-
From: Tommaso Teofili [mailto:tommaso.teof...@gmail.com]
Sent: Tuesday, January 27, 2015 11:17 PM
To: dev@hama.apache.org
Subject: Re: Q, OffHeapVerticesInfo, CacheService.free() method for remove
vertex
I've fixed it in HAMA-925,
Regards,
Tommaso
2015-01-27 8:50 GMT+01:00 Tommaso Teofili tommaso.teof...@gmail.com:
if I recall correctly the VerticesInfo#removeVertex API was not used at
the time I implemented the OffHeapVerticesInfo, that's why it was not
implemented, however if you refer to [1] I'd say that's surely a good idea
to implement it using that API.
Regards,
Tommaso
[1] :
http://directmemory.apache.org/apidocs/reference/org/apache/directmemory/cache/CacheService.html#free(K)
2015-01-27 6:00 GMT+01:00 Edward J. Yoon edward.y...@samsung.com:
Hi,
It seems CacheService.free() method can be used to remove vertex.
Is there a specific reason why you throw UnsupportedOperationException?
@Override
public void removeVertex(V vertexID) {
throw new UnsupportedOperationException(Not yet implemented);
}
--
Best Regards, Edward J. Yoon
[jira] [Commented] (HAMA-925) OffHeapVerticesInfo should implement removeVertex
[ https://issues.apache.org/jira/browse/HAMA-925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14294421#comment-14294421 ] Hudson commented on HAMA-925: - SUCCESS: Integrated in Hama-Nightly-for-Hadoop-2.x #501 (See [https://builds.apache.org/job/Hama-Nightly-for-Hadoop-2.x/501/]) HAMA-925 - added OffHeapVerticesInfo#removeVertex and #finishRemovals implementations (tommaso: rev 1655029) * /hama/trunk/graph/src/main/java/org/apache/hama/graph/OffHeapVerticesInfo.java OffHeapVerticesInfo should implement removeVertex - Key: HAMA-925 URL: https://issues.apache.org/jira/browse/HAMA-925 Project: Hama Issue Type: Improvement Components: graph Affects Versions: 0.6.4 Reporter: Tommaso Teofili Assignee: Tommaso Teofili Fix For: 0.7.0 As brought up by [~udanax] and discussed on the ML OffHeapVerticesInfo#removeVertex should call CacheService#free to properly remove the passed vertex from DM. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
Jenkins build is back to normal : Hama-Nightly-for-Hadoop-2.x #501
See https://builds.apache.org/job/Hama-Nightly-for-Hadoop-2.x/501/changes
Re: Legal question to include code like com.sun.tools.javac.Main
A better alternative would be to use the Eclipse compiler, which you can easily add to your distribution as a jar file (or set of jar files). Like Apache Tomcat does. By doing so, you would also enable to use a simple JRE for running Hama, rather than depending on a full blown JDK. Not to mention other advantages, like independence of a particular Java version. Jochen On Mon, Jan 26, 2015 at 10:45 AM, Chia-Hung Lin cli...@googlemail.com wrote: Hi, I have a naive question regarding to include the methods like Main.compile in com.sun package in the project source code. For instance, in our project like hama if there is a source file that makes use of com.sun.tools.javac.Main to runtime compile java sources into classes. Is it legal to release or include that source with project? Thanks - To unsubscribe, e-mail: legal-discuss-unsubscr...@apache.org For additional commands, e-mail: legal-discuss-h...@apache.org -- Our time is just a point along a line that runs forever with no end. (Al Stewart, Lord Grenville)
Re: Legal question to include code like com.sun.tools.javac.Main
Thanks for the information. It's very clear. The last question. Scala's licensed under [1], so assuming it's legal to use it's nsc (compiler package) code because its license is not listed in [2] and does not violate Apache's. Am I correct in that assumption? Thanks again for kindly help! [1]. http://www.scala-lang.org/license.html [2]. http://www.apache.org/legal/resolved.html#category-x On 27 January 2015 at 18:43, Mark Thomas ma...@apache.org wrote: On 26/01/2015 09:45, Chia-Hung Lin wrote: Hi, I have a naive question regarding to include the methods like Main.compile in com.sun package in the project source code. For instance, in our project like hama if there is a source file that makes use of com.sun.tools.javac.Main to runtime compile java sources into classes. Is it legal to release or include that source with project? Legally, yes that is fine. You can reference internal JVM vendor classes if you wish. What you may not do is include tools.jar in your distribution since the license for that JAR is not compatible with distribution under the ALv2. Mark - To unsubscribe, e-mail: legal-discuss-unsubscr...@apache.org For additional commands, e-mail: legal-discuss-h...@apache.org
[jira] [Resolved] (HAMA-925) OffHeapVerticesInfo should implement removeVertex
[ https://issues.apache.org/jira/browse/HAMA-925?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tommaso Teofili resolved HAMA-925. -- Resolution: Fixed fixed in r1655029 (also added implementation for VerticesInfo#finishRemovals via CacheService#collectExpired). OffHeapVerticesInfo should implement removeVertex - Key: HAMA-925 URL: https://issues.apache.org/jira/browse/HAMA-925 Project: Hama Issue Type: Improvement Components: graph Affects Versions: 0.6.4 Reporter: Tommaso Teofili Assignee: Tommaso Teofili Fix For: 0.7.0 As brought up by [~udanax] and discussed on the ML OffHeapVerticesInfo#removeVertex should call CacheService#free to properly remove the passed vertex from DM. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HAMA-925) OffHeapVerticesInfo should implement removeVertex
[ https://issues.apache.org/jira/browse/HAMA-925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14293488#comment-14293488 ] Hudson commented on HAMA-925: - SUCCESS: Integrated in Hama-trunk #338 (See [https://builds.apache.org/job/Hama-trunk/338/]) HAMA-925 - added OffHeapVerticesInfo#removeVertex and #finishRemovals implementations (tommaso: rev 1655029) * /hama/trunk/graph/src/main/java/org/apache/hama/graph/OffHeapVerticesInfo.java OffHeapVerticesInfo should implement removeVertex - Key: HAMA-925 URL: https://issues.apache.org/jira/browse/HAMA-925 Project: Hama Issue Type: Improvement Components: graph Affects Versions: 0.6.4 Reporter: Tommaso Teofili Assignee: Tommaso Teofili Fix For: 0.7.0 As brought up by [~udanax] and discussed on the ML OffHeapVerticesInfo#removeVertex should call CacheService#free to properly remove the passed vertex from DM. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Created] (HAMA-925) OffHeapVerticesInfo should implement removeVertex
Tommaso Teofili created HAMA-925: Summary: OffHeapVerticesInfo should implement removeVertex Key: HAMA-925 URL: https://issues.apache.org/jira/browse/HAMA-925 Project: Hama Issue Type: Improvement Components: graph Affects Versions: 0.6.4 Reporter: Tommaso Teofili Assignee: Tommaso Teofili Fix For: 0.7.0 As brought up by [~udanax] and discussed on the ML OffHeapVerticesInfo#removeVertex should call CacheService#free to properly remove the passed vertex from DM. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
