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

Crawled data graph #130

Open
wants to merge 47 commits into
base: master
Choose a base branch
from
Open

Conversation

param-jot
Copy link
Contributor

The test case is not correctly implemented yet. But the code is working.

sritejakv and others added 30 commits April 8, 2019 15:33
Merging develop to master
…y and changes in frontier-context.xml to set env variables
… changes in SPARQL query to retreive endTimeStamps
@MichaelRoeder MichaelRoeder changed the base branch from master to develop January 8, 2020 16:39
@MichaelRoeder MichaelRoeder changed the base branch from develop to master January 8, 2020 16:40
Copy link
Member

@MichaelRoeder MichaelRoeder left a comment

Choose a reason for hiding this comment

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

Please have a look at the comments I left.


import org.dice_research.squirrel.data.uri.CrawleableUri;

public interface OutDatedUriRetriever extends Closeable {
Copy link
Member

Choose a reason for hiding this comment

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

Please add a Javadoc that describes the functionality of this interface.

import java.util.ArrayList;
import java.util.List;

public class SparqlhostConnector implements OutDatedUriRetriever {
Copy link
Member

Choose a reason for hiding this comment

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

Please add a Javadoc comment describing the functionality of this class.

Please choose a better name for this class, e.g., SparqlBasedOutDatedUriRetriever

@param-jot
Copy link
Contributor Author

Hi, This branch is not updated with SPARQL recrawling changes. It's only for crawledDataGraph module. I created a different branch for recrawling feature. I don't know why it is mixed up. I'll send a different PR for recrawling. Thanks!

@MichaelRoeder
Copy link
Member

You already have this PR: #126

@param-jot
Copy link
Contributor Author

Yes.

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.

3 participants