Skip to content
This repository has been archived by the owner on Jan 13, 2022. It is now read-only.

create Apache Giraph-compatible facebook jars. #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ekoontz
Copy link

@ekoontz ekoontz commented Jun 29, 2012

Giraph (http://giraph.apache.org)'s facebook profile has the following:

<dependency>
      <groupId>com.facebook.hadoop</groupId>
      <artifactId>hadoop-core</artifactId>
      <version>0.20.1</version>
      <type>jar</type>
      <scope>system</scope>
      <systemPath>${hadoop.jar.path}</systemPath>
</dependency>

However Facebook's build.xml and copy-hdfs-jars-to-maven.sh do not conform to this.

@ghost ghost assigned aching Jun 29, 2012
@weiyanwang
Copy link
Contributor

@aching could you take a look? I am not familiar with giraph. Thanks!

@aching
Copy link
Contributor

aching commented Jul 2, 2012

Eugene, thanks for looking into this. I think the dependency should rather be org.apache.hadoop instea od com.facebook.hadoop. My bad.

@nitay
Copy link

nitay commented Oct 20, 2012

@aching can we revisit this? I disagree with you, I think the groupId should definitely be com.facebook.hadoop as @ekoontz has it. In fact if you look at the main maven repo there are a few groups that publish hadoop jars (http://search.maven.org/#search%7Cga%7C1%7Chadoop-core). That is the whole point of setting the groupId to the company so you know who is providing it.

Anyways, can we get this in? I this we should get the facebook jar published all the way up to maven. See e.g. https://docs.sonatype.org/display/Repository/Sonatype+OSS+Maven+Repository+Usage+Guide. I can take this on this step after we get this diff in. Then the whole hack with user specifying path to jar can be taken out altogether and people will be able to build off of Facebook's hadoop just as easily as any other profile in Giraph.

Thoughts?

@aching
Copy link
Contributor

aching commented Oct 20, 2012

That seems reasonable. Let me ask around on Monday and see if anyone uses this script.

@aching aching closed this Oct 20, 2012
@aching aching reopened this Oct 20, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants