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

Improve description of database and add - import part #17

Merged
merged 3 commits into from
Mar 6, 2017

Conversation

gpetit
Copy link
Contributor

@gpetit gpetit commented Mar 6, 2017

Improve description of database : add a schema

Improve description for "add - import" part (thx @nicolas-f ) as asked in orbisgis/orbisgis#1193

@gpetit gpetit requested a review from ebocher March 6, 2017 11:35
@@ -71,7 +71,11 @@ Database type:
* H2GIS_SERVER: remote H2GIS db,
* POSTGIS: remote PostGIS db.

The figure below summarizes this philosophy:
Copy link
Member

Choose a reason for hiding this comment

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

architecture ? or usage ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok for usage

* Import: import a file and convert it into a table.
* Add: Link a new database table to a file (vector, alphanumeric, ...),
* Pro:
* Immediate table creation,
Copy link
Member

Choose a reason for hiding this comment

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

Direct link to the data

* Add: Link a new database table to a file (vector, alphanumeric, ...),
* Pro:
* Immediate table creation,
* Does not take more disk space,
Copy link
Member

Choose a reason for hiding this comment

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

remove more

* Cons:
* Read-only table,
* Cannot create alpha-numeric index,
* Only compatible with h2 database.
Copy link
Member

@ebocher ebocher Mar 6, 2017

Choose a reason for hiding this comment

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

Nope we can connect two remote Postgis tables with dblink https://www.postgresql.org/docs/9.2/static/dblink.html

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok so I remove the last line

@nicolas-f
Copy link
Member

The add function does not support dblink

@nicolas-f
Copy link
Member

And dblink does not connect to a file but to another database

@nicolas-f
Copy link
Member

nicolas-f commented Mar 6, 2017

So the Add function is only compatible with local h2 database

@ebocher
Copy link
Member

ebocher commented Mar 6, 2017

@nicolas-f
You are right we will support dblink and datalink in the future from a WPS script

@ebocher
Copy link
Member

ebocher commented Mar 6, 2017

Nice

@ebocher ebocher merged commit c04b2b9 into orbisgis:master Mar 6, 2017
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

3 participants