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

Some tips #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 22 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ java -jar $ili2pg --schemaimport \
--createMetaInfo \ # creates additional tables with information from the interlis model
--createNumChecks \ # creates constraints for numerical data types
--createUnique \ # creates unique constraints in the db for Interlis unique constraints
--createImportTabs \ # creates import tables for trace import operations
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would put this option under "Options that might be relevant dependent on individual settings"

--expandMultilingual \ # LocalisationCH_V1.MultilingualText/MText --> additional columns in table
--expandLocalised \ # LocalisationCH_V1.LocalisedText/MText --> additional columns in table
--setupPgExt \ # PostGIS: erstellt postgreql Erweiterungen 'uuid-ossp' und 'postgis' (falls noch nicht vorhanden)
Expand Down Expand Up @@ -108,6 +109,26 @@ java -jar $ili2pg --import \
interlis.xtf
```

### Data replace

**Selected options for ili2pg**

```
ili2pg=ili2pg-4.6.0.jar

java -jar $ili2pg --replace \
--dbhost $PGHOST \
--dbport $PGPORT \
--dbdatabase $PGDB \
--dbusr $PGUSER \
--dbpwd $PGPASSWORD \
--dbschema $SCHEMA_NAME \ # e.g. "motorways_project_planing_zones"
--strokeArcs \
--dataset $var_dataset \ # dataset name
interlis.xtf
```


### Additional tables required for pyramid_oereb which are not part of OeREBKRMtrsfr_V2_0

**add_tables_to_trsf_structure.sql**
Expand All @@ -121,11 +142,7 @@ CREATE TABLE IF NOT EXISTS :schema.datenintegration
datum timestamp without time zone NOT NULL,
amt bigint NOT NULL,
checksum character varying COLLATE pg_catalog."default",
CONSTRAINT datenintegration_pkey PRIMARY KEY (t_id),
CONSTRAINT datenintegration_amt_fkey FOREIGN KEY (amt)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Instead of removing this, I would suggest to add a comment that the constraint "datenintegration_amt_fkey" might raise conflicts if a dataset is replaced by ili2pg and, therefore, a user has to decide if he wants to ommit it.

REFERENCES :schema.amt (t_id) MATCH SIMPLE
ON UPDATE NO ACTION
ON DELETE NO ACTION
CONSTRAINT datenintegration_pkey PRIMARY KEY (t_id)
)
WITH (
OIDS = FALSE
Expand Down Expand Up @@ -167,4 +184,3 @@ psql -d $PGDB -U $PGUSER -v "usr=$PGUSER" -v "schema=$SCHEMA_NAME" -f add_tables
psql -d $PGDB -U $PGUSER -v "schema=$SCHEMA_NAME" -f update_availability.sql

```