[jira] [Commented] (HAMA-989) Build fails on non-Linux systems

[Edward J. Yoon (JIRA)]

    [ 
https://issues.apache.org/jira/browse/HAMA-989?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15257343#comment-15257343
 ] 

Edward J. Yoon commented on HAMA-989:
-------------------------------------

We can catch and ignore exceptions or, SystemUtils.

{code}
diff --git 
a/core/src/test/java/org/apache/hama/bsp/message/TestHamaAsyncMessageManager.java
 
b/core/src/test/java/org/apache/hama/bsp/message/TestHamaAsyncMessageManager.java
index f4f89b9..b7bc9c8 100644
--- 
a/core/src/test/java/org/apache/hama/bsp/message/TestHamaAsyncMessageManager.java
+++ 
b/core/src/test/java/org/apache/hama/bsp/message/TestHamaAsyncMessageManager.java
@@ -23,6 +23,7 @@
 
 import junit.framework.TestCase;
 
+import org.apache.commons.lang.SystemUtils;
 import org.apache.hadoop.fs.FileSystem;
 import org.apache.hadoop.io.IntWritable;
 import org.apache.hadoop.io.NullWritable;
@@ -45,10 +46,14 @@
   public static volatile int increment = 1;
 
   public void testMemoryMessaging() throws Exception {
-    HamaConfiguration conf = new HamaConfiguration();
-    conf.setClass(MessageManager.RECEIVE_QUEUE_TYPE_CLASS, MemoryQueue.class,
-        MessageQueue.class);
-    messagingInternal(conf);
+    if (SystemUtils.IS_OS_LINUX) {
+      HamaConfiguration conf = new HamaConfiguration();
+      conf.setClass(MessageManager.RECEIVE_QUEUE_TYPE_CLASS, MemoryQueue.class,
+          MessageQueue.class);
+      messagingInternal(conf);
+    } else {
+      // we skip this test bc AsyncRPC is currently support only linux
+    }
   }
 
   private static void messagingInternal(HamaConfiguration conf)
{code}

WDYT?

> Build fails on non-Linux systems
> --------------------------------
>
>                 Key: HAMA-989
>                 URL: https://issues.apache.org/jira/browse/HAMA-989
>             Project: Hama
>          Issue Type: Bug
>          Components: bsp core, build 
>    Affects Versions: 0.7.1
>            Reporter: Edward J. Yoon
>             Fix For: 0.7.2
>
>
> http://markmail.org/message/ipgc5fjs57xdmtr2



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)