Skip to content
This repository was archived by the owner on Jul 28, 2025. It is now read-only.

added zpm support#2

Open
SergeyMi37 wants to merge 2 commits intobdeboe:masterfrom
SergeyMi37:master
Open

added zpm support#2
SergeyMi37 wants to merge 2 commits intobdeboe:masterfrom
SergeyMi37:master

Conversation

@SergeyMi37
Copy link

No description provided.

@bdeboe
Copy link
Owner

bdeboe commented Sep 2, 2022

@SergeyMi37 can you please add the logic to set up the REST app that's currently in Demo.SetAnalysis.Setup:CreateRestWebApp() to the module definition? Also, the Demo.Hotels.Tables.* classes are projections created and managed by the iKnow domain definition. They should not be part of this repo. You could even leave the whole Demo.Hotels tree out of the zpm part, as it doesn't come with sample data and is not needed for using the Set Analysis app.

Thanks!

@SergeyMi37
Copy link
Author

SergeyMi37 commented Sep 2, 2022

If you do not include the Demo.Hotels.Tables.* package, then an error will be generated during installation:

...
[isc-iknow-setanalysis] Reload SUCCESS
[isc-iknow-setanalysis] Module object refreshed.
[isc-iknow-setanalysis] Validate START
[isc-iknow-setanalysis] Validate SUCCESS
[isc-iknow-setanalysis] Compile START
[isc-iknow-setanalysis] Compile FAILURE
ERROR! Storage on class Demo.Hotels.Tables.CcUnique modified by storage compiler, developer should have run ^build to make sure all storage is updated correctly and saved to Perforce

This is a feature of the current implementation of zpm.
I was forced to include this package in the repo.

@bdeboe
Copy link
Owner

bdeboe commented Sep 2, 2022

That's due to a stale reference in the Extent Manager. We've seen this in some cases when programmatically deleting these projections other than through the domain definition's machinery. This should pass without error in an empty namespace, and you should be able to recover from it in an existing one by using %ExtentMgr.Util:DeleteExtentDefinitionIfExists() for the extents (class names) it complains about.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants