Skip to content

missing changed storage info when compiling multiple classes #1537

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

Closed
dkutac opened this issue Apr 23, 2025 · 5 comments · Fixed by #1538
Closed

missing changed storage info when compiling multiple classes #1537

dkutac opened this issue Apr 23, 2025 · 5 comments · Fixed by #1538
Assignees
Labels
bug Something isn't working

Comments

@dkutac
Copy link

dkutac commented Apr 23, 2025

Type: Bug

when compiling a class which inherits from other classes, sometimes it is possible that compilation modifies storage of those other classes. However, these changes are not sent back from IRIS server to local files, thus a discrepancy occurs between local files and their server copies. This behaviour is leading to issues when using GIT (or other source control) and deployment of changes to other IRIS server(s) as compilation on those other servers may generate different storage changes to dependent classes.

Extension version: 3.0.1
VS Code version: Code 1.99.3 (17baf841131aa23349f217ca7c570c76ee87b957, 2025-04-15T23:18:46.076Z)
OS version: Windows_NT x64 10.0.26100
Modes:

@isc-bsaviano
Copy link
Contributor

@dkutac This has always been the behavior. The extension only loads changes for the files that have been compiled, not any "related" documents. I'm not sure what we can realistically do about this. There are a few questions that would have to be answered for any file that was compiled:

  1. What non-Generated files may have changed due to to the compilation of this file?
  2. Did any of those files actually change?
  3. Do those files need to be recompiled for those changes to take effect before the changes can be exported?

@isc-bsaviano
Copy link
Contributor

The Atelier APIs that we use for compilation do return an array of documents updated by the compile, so this should be possible. @dkutac If would appreciate it if you could send me a few test classes on here or internally so I can work on this.

@isc-bsaviano isc-bsaviano added the enhancement New feature or request label Apr 24, 2025
@isc-bsaviano isc-bsaviano self-assigned this Apr 24, 2025
@dkutac
Copy link
Author

dkutac commented Apr 24, 2025

I'll talk to customer who initiated this and get some test classes in next couple days.

@zvondic
Copy link

zvondic commented Apr 25, 2025

NNH.Test.zip
@dkutac In VSC:

  1. Add some property to ClassA.
  2. Compile ClassP with flags ckb
  3. Classes Class1 and Class2 don't have good storage in locale file storage but i can see god storage on Cache server side

@isc-bsaviano
Copy link
Contributor

Thanks, I will take a look.

@isc-bsaviano isc-bsaviano added bug Something isn't working and removed enhancement New feature or request labels Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants