Skip to content
This repository has been archived by the owner on Dec 27, 2021. It is now read-only.

The metamodel doesn't use EReferences #71

Open
wpiers opened this issue Sep 6, 2021 · 1 comment
Open

The metamodel doesn't use EReferences #71

wpiers opened this issue Sep 6, 2021 · 1 comment

Comments

@wpiers
Copy link
Collaborator

wpiers commented Sep 6, 2021

EReferences are not in use in the metamodel, because of legacy choices :

  • [Xtext] For scalability reasons we want to exploit modularity and decompose POOSL models in multiple textual files. After consultation of Itemis at XTextCon’15, we are using the indexer and rely on the NodeModelUtils to get the textual value of a reference. To avoid loading many files into memory when opening just one file, we have to make sure not to perform a get-operation on an EReference-attribute (e.g., to get the textual name of the referred element).

  • [Sirius] Repeatedly experiencing problems with serialization from Sirius (beta users were regularly unable to save their models after editing).

It will lead us to serious maintenance issues, whether the Sirius/XText bridge could help on this issue should be investigated.
The use of the org.eclipse.poosl.xtext.custom.PooslCacheEntry cache should be avoided.

@wpiers wpiers added this to the M3 milestone Sep 6, 2021
@mypsycho
Copy link
Collaborator

This is probably related to issue with Sirius/xtext architecture #59

@wpiers wpiers added the P2 label Oct 13, 2021
@mypsycho mypsycho removed this from the M3 milestone Oct 18, 2021
@wpiers wpiers removed the P2 label Nov 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants