Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add reindexing service #28

Closed
wants to merge 2 commits into from
Closed

Add reindexing service #28

wants to merge 2 commits into from

Conversation

acoburn
Copy link
Contributor

@acoburn acoburn commented May 27, 2015

See: https://jira.duraspace.org/browse/FCREPO-1488

This creates a fcrepo-reindexing module in fcrepo-camel-toolbox

@awoods
Copy link

awoods commented May 28, 2015

@acoburn
Copy link
Contributor Author

acoburn commented May 28, 2015

Yes, they do. Thanks for catching that!

@@ -33,6 +33,11 @@
<groupId>org.apache.camel</groupId>
<artifactId>camel-http4</artifactId>
</dependency>

<dependency>
<groupId>org.apache.camel</groupId>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not seeing where camel-restlet is used in fcrepo-indexing-solr.
I removed this dependency locally, and the build completes successfully.
If there is no particular reason for this dependency here, let's drop it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, that should be removed. That's from when I started implementing the reindexing piece inside the indexing-solr application. I have a number of changes that are part of a forthcoming commit.

remove activemq endpoints from unit tests
@awoods
Copy link

awoods commented May 29, 2015

Resolved with: 170c289

@awoods awoods closed this May 29, 2015
@acoburn acoburn deleted the fcrepo-1488 branch August 17, 2015 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants