Skip to content

fix: include reuse-module initial data in @cap-js/postgres build - #1707

Open
alex-bkmr-cbs wants to merge 1 commit into
cap-js:mainfrom
alex-bkmr-cbs:fix/postgres-reuse-module-initial-data-1694
Open

fix: include reuse-module initial data in @cap-js/postgres build#1707
alex-bkmr-cbs wants to merge 1 commit into
cap-js:mainfrom
alex-bkmr-cbs:fix/postgres-reuse-module-initial-data-1694

Conversation

@alex-bkmr-cbs

Copy link
Copy Markdown

The postgres build task copied only the app-local db/data folder, dropping
initial-data CSVs shipped by reuse packages from the gen/pg artifact. This made
a fresh Postgres deployment inconsistent with cds deploy from source and with
the HANA build task.

Collect initial data across app + reuse modules via cds.deploy.resources() and
write the merged per-entity union into gen/pg/db/data. Gated by a csvFileDetection
option (default on) for parity with HANA; setting it to false restores the legacy
local-only copy.

Closes #1694

Signed-off-by: Alex alexander.buchkremer@cbs-consulting.de

The postgres build task copied only the app-local db/data folder, dropping
initial-data CSVs shipped by reuse packages from the gen/pg artifact. This made
a fresh Postgres deployment inconsistent with `cds deploy` from source and with
the HANA build task.

Collect initial data across app + reuse modules via cds.deploy.resources() and
write the merged per-entity union into gen/pg/db/data. Gated by a csvFileDetection
option (default on) for parity with HANA; setting it to false restores the legacy
local-only copy.

Closes cap-js#1694

Signed-off-by: Alex <alexander.buchkremer@cbs-consulting.de>
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.

@cap-js/postgres build omits reuse-module (dependency) initial data from gen/pg

1 participant