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

Implement loading to staging/live virtuoso #104

Closed
Bjwebb opened this issue Sep 8, 2015 · 3 comments
Closed

Implement loading to staging/live virtuoso #104

Bjwebb opened this issue Sep 8, 2015 · 3 comments
Assignees

Comments

@Bjwebb
Copy link
Member

Bjwebb commented Sep 8, 2015

No description provided.

@Bjwebb
Copy link
Member Author

Bjwebb commented Sep 9, 2015

The original plan was to using pyodbc for this, and I even updated the Virtuoso patch to support v3 Bjwebb/pyodbc@459382d, and I got as far as acquiring a cursor - NRGI/resource-projects-etl@8db4a99 - however, using that cursor caused errors.

Instead, I'm now trying to add sparql using virtuoso's sparql-graph-crud-auth web endpoint. I also had trouble using "requests" for this (see 96fa2fc#diff-d6efc1c3c41753d42e33120dc96d54bcR73 for the erroring code). So for now I'm using subprocess to talk to curl.

This now successfully loads into virtuoso.

Some of the remaining tasks:

@Bjwebb
Copy link
Member Author

Bjwebb commented Sep 10, 2015

I've looked into graph groups, and as far as I can tell this involves running SQL rather than SPARQL commands. There is no nice REST endpoint for SQL that I can see. Thus I think we have 3 options:

  1. Get pyodbc working - not sure how much work this is, but my exprience so far suggests it won't be straightforward.
  2. Do the cookie based auth to get at http://localhost:8890/conductor/isql_main.vspx over HTTP - this is inelegant and brittle
  3. Call virtuoso's isql command using subprocess. This requires running Cove in an environment that has access to virtuoso's isql - e.g. by building it in a container inherited from the virtuoso one.
    I'm investigating option (3) for now.

@timgdavies
Copy link
Contributor

+1 to Option 3.

Bjwebb added a commit to NRGI/resource-projects-etl that referenced this issue Sep 10, 2015
(This still inherits from the virtuoso container to ensure that cove has
access to a working isql command).
Bjwebb added a commit to NRGI/resource-projects-etl that referenced this issue Oct 5, 2015
Bjwebb added a commit to NRGI/resource-projects-etl that referenced this issue Oct 5, 2015
Bjwebb added a commit to NRGI/resource-projects-etl that referenced this issue Oct 5, 2015
@Bjwebb Bjwebb closed this as completed Oct 5, 2015
@Bjwebb Bjwebb removed the 3 - Done label Oct 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants