tanishqgandhi1908 commented on code in PR #7789:
URL: https://github.com/apache/texera/pull/7789#discussion_r3825964254
##########
sql/updates/36.sql:
##########
@@ -23,20 +23,28 @@ SET search_path TO texera_db;
BEGIN;
--- The file resolver now requires an explicit resource-type prefix on dataset
--- logical paths (/datasets/ownerEmail/datasetName/versionName/...) so other
--- resource types (e.g. models) can be told apart by the prefix. Existing
--- workflows store unprefixed dataset paths inside workflow.content and
--- workflow_version.content, in two operator properties:
+-- The file resolver requires an explicit resource-type prefix on dataset
logical
Review Comment:
Deliberate. changeSet 36 is not in any released version: it's absent from
release/v1.2, whose changelog tops out at id="1", and no tag contains it. So no
release has a stored checksum to invalidate.
That leaves developer machines and deployments tracking main, and you're
right that clearing the checksum there would skip the normalization. Those are
handled by running the file by hand:
psql -h localhost -U postgres -f sql/updates/36.sql
It's idempotent and covers all three shapes — unprefixed, plural-prefixed,
and already-current — so it converges whatever state the database is in. Adding
a post-37 changeSet was the original approach here; it was dropped in favour of
correcting 36 in place, since the prefix it introduced has never shipped and two
migrations describing one path format is worse to carry forward.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
