You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bigtop.apache.org by GitBox <gi...@apache.org> on 2021/03/09 12:45:16 UTC
[GitHub] [bigtop] iwasakims opened a new pull request #753: BIGTOP-3512. Drop Giraph.
iwasakims opened a new pull request #753:
URL: https://github.com/apache/bigtop/pull/753
https://issues.apache.org/jira/browse/BIGTOP-3512
Drop Giraph based on the discussion in [BIGTOP-3471](https://issues.apache.org/jira/browse/BIGTOP-3471).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [bigtop] iwasakims commented on pull request #753: BIGTOP-3512. Drop Giraph.
Posted by GitBox <gi...@apache.org>.
iwasakims commented on pull request #753:
URL: https://github.com/apache/bigtop/pull/753#issuecomment-793845420
I ran following on CentOS 8 in order to check that the code removal does not harm manifests and tests of other components.
```
$ ./gradlew hadoop-clean hadoop-pkg repo
$ cd provisioner/docker
$ ./docker-hadoop.sh \
--create 3 \
--image bigtop/puppet:trunk-centos-8 \
--memory 8g \
--repo file:///bigtop-home/output \
--disable-gpg-check \
--stack hdfs,mapreduce,yarn \
--smoke-tests hdfs,mapreduce,yarn
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [bigtop] sekikn commented on pull request #753: BIGTOP-3512. Drop Giraph.
Posted by GitBox <gi...@apache.org>.
sekikn commented on pull request #753:
URL: https://github.com/apache/bigtop/pull/753#issuecomment-794562075
After applying this PR, it seems that we still have some giraph-related configurations in bigtop.bom and pom.xml.
```
$ curl -sL https://github.com/apache/bigtop/pull/753.patch | git apply -
$ git grep -i giraph
...
bigtop.bom: So, below phoenix and giraph ought to be build after the hbase has been built
bigtop.bom: 'oozie', 'flume', 'giraph', 'solr', 'spark',
bigtop.bom: hbase:['phoenix','giraph','ycsb','hive'],
bigtop.bom: 'giraph' {
bigtop.bom: name = 'giraph'
bigtop.bom: relNotes = 'Apache Giraph'
pom.xml: <giraph.version>1.1.0</giraph.version>
pom.xml: <groupId>org.apache.giraph</groupId>
pom.xml: <artifactId>giraph</artifactId>
pom.xml: <version>${giraph.version}</version>
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [bigtop] sekikn commented on pull request #753: BIGTOP-3512. Drop Giraph.
Posted by GitBox <gi...@apache.org>.
sekikn commented on pull request #753:
URL: https://github.com/apache/bigtop/pull/753#issuecomment-794613652
+1, confirmed that some components are successfully built after applying this PR.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [bigtop] iwasakims commented on pull request #753: BIGTOP-3512. Drop Giraph.
Posted by GitBox <gi...@apache.org>.
iwasakims commented on pull request #753:
URL: https://github.com/apache/bigtop/pull/753#issuecomment-794605277
Oops. I forgot to add changes in top directory. updated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [bigtop] sekikn commented on pull request #753: BIGTOP-3512. Drop Giraph.
Posted by GitBox <gi...@apache.org>.
sekikn commented on pull request #753:
URL: https://github.com/apache/bigtop/pull/753#issuecomment-794630849
Merged into master.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [bigtop] sekikn merged pull request #753: BIGTOP-3512. Drop Giraph.
Posted by GitBox <gi...@apache.org>.
sekikn merged pull request #753:
URL: https://github.com/apache/bigtop/pull/753
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org