giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jakob Homan (Commented) (JIRA)" <>
Subject [jira] [Commented] (GIRAPH-168) Simplify munge directive usage with new munge flag HADOOP_SECURE (rather than HADOOP_FACEBOOK) and remove usage of HADOOP
Date Wed, 04 Apr 2012 17:09:29 GMT


Jakob Homan commented on GIRAPH-168:

My understanding was that the RPC changes FB had made were backports of changes that are in
later versions, so I'm not sure if OldRPC is the correct description.  Also, within the Hadoop
world there's not really talk of old versus new RPC (except for the PB-based stuff, which
will make this really confusing...).  Hadoop security is API-incompatible with Hadoop non-security
(due to changes in UGI) and FB's distro is insecure and API incompatible due to new APIs backported
from more modern versions.
> Simplify munge directive usage with new munge flag HADOOP_SECURE (rather than HADOOP_FACEBOOK)
and remove usage of HADOOP
> -------------------------------------------------------------------------------------------------------------------------
>                 Key: GIRAPH-168
>                 URL:
>             Project: Giraph
>          Issue Type: Improvement
>    Affects Versions: 0.2.0
>            Reporter: Eugene Koontz
>            Assignee: Eugene Koontz
>         Attachments: GIRAPH-168.patch, GIRAPH-168.patch, GIRAPH-168.patch
> This JIRA relates to the mail thread here: 
> Currently we check for the munge flags HADOOP, HADOOP_FACEBOOK and HADOOP_NON_SECURE
when using munge in a few places. Hopefully we can eliminate usage of munge in the future,
but until then, we can mitigate the complexity by consolidating the number of flags checked.
This JIRA renames HADOOP_FACEBOOK to HADOOP_SECURE, and removes usages of HADOOP, to handle
the same conditional compilation requirements. It also makes it easier to add more maven profiles
so that we can easily increase our hadoop version coverage.
> This patch modifies the existing hadoop_facebook profile to use the new HADOOP_SECURE
munge flag, rather than HADOOP_FACEBOOK.
> It also adds a new hadoop maven profile, hadoop_trunk, which also sets HADOOP_SECURE.

> Finally, it adds a default profile, hadoop_0.20.203. This is needed so that we can specify
its dependencies separately from hadoop_trunk, because the hadoop dependencies have changed
between trunk and 0.205.0 - the former requires hadoop-common, hadoop-mapreduce-client-core,
and hadoop-mapreduce-client-common, whereas the latter requires hadoop-core. 
> With this patch, the following passes:
> {code}
> mvn clean verify && mvn -Phadoop_trunk clean verify && mvn -Phadoop_0.20.203
clean verify
> {code}
> Current problems: 
> * I left in place the usage of HADOOP_NON_SECURE, but note that the profile that uses
this is hadoop_non_secure, which fails to compile on trunk:
> * I couldn't get -Phadoop_facebook to work; does this work outside of Facebook?

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


View raw message