Skip to content
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.

Sign up for GitHub

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

Jump to bottom

EntityTypeService.readAllEntityTypes makes one webapi call per each etype #3

Open
DavidLeoni opened this issue Jul 10, 2014 · 3 comments
Open

EntityTypeService.readAllEntityTypes makes one webapi call per each etype #3

DavidLeoni opened this issue Jul 10, 2014 · 3 comments
Assignees
@ITankoyeu
Labels
enhancement EtypeService performance

Comments

@DavidLeoni
Copy link
Member

In order to have attribute defs inside the etypes, EntityTypeService.getAllEntityTypescalls AttributeDefinitionClient.readAttributeDefinitions() for each etype, thus making one webapi call per each etype which is very slow
@ITankoyeu
Copy link
Contributor

I have checked the time needed for getting all etypes with all attrdefs (currently 25 etypes). It takes only 2176 miliseconds (~2 seconds only). Unfortunatelly, get them with one call is not supported yet from the server side.

@DavidLeoni
Copy link
Member Author

It takes only 2176 miliseconds

It takes that much when server is behaving properly. Yesterday Enrico made some adjustment to Tomcat - let's hope performance keeps that good.

@ITankoyeu
Copy link
Contributor

So... we need to make sure that server is behaving properly :)

@DavidLeoni DavidLeoni changed the title EntityTypeService.getAllEntityTypes makes one webapi call per each etype EntityTypeService.readAllEntityTypes makes one webapi call per each etype Mar 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ITankoyeu ITankoyeu

Labels
enhancement EtypeService performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@DavidLeoni @ITankoyeu