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

Remove workaround and extra verification for UIMA-6162 #3116

Closed
reckart opened this issue Jun 12, 2022 · 0 comments
Closed

Remove workaround and extra verification for UIMA-6162 #3116

reckart opened this issue Jun 12, 2022 · 0 comments
Assignees
@reckart
Labels
🚀 Performance Performance / scalability ⚙️ Refactoring
Projects
Kanban
Milestone
24.0

Comments

@reckart
Copy link
Member

reckart commented Jun 12, 2022

Describe the refactoring action
Because of UIMA-6162, we had introduced an extra serialization and deserialization step before persisting the CAS actually to disk. However, UIMA-6162 has been fixed since a while and the extra
verification is taking up time in particular for larger CASes.

Expected benefit
To improve reaction times for large CASes, the extra verification should be dropped.
@reckart reckart added ⚙️ Refactoring 🚀 Performance Performance / scalability labels Jun 12, 2022
@reckart reckart added this to the 24.0 milestone Jun 12, 2022
@reckart reckart self-assigned this Jun 12, 2022
@reckart reckart added this to 🔖 To do in Kanban via automation Jun 12, 2022
reckart added a commit that referenced this issue Jun 12, 2022
@reckart
#3116 - Remove workaround and extra verification for UIMA-6162
ba42e0c 
- Remove extra verification
- Add property to re-enable extra verification
reckart added a commit that referenced this issue Jun 12, 2022
@reckart
Merge pull request #3117 from inception-project/refactoring/3116-Remo…
de1bf76 
…ve-workaround-and-extra-verification-for-UIMA-6162

#3116 - Remove workaround and extra verification for UIMA-6162
@reckart reckart closed this as completed Jun 12, 2022
Kanban automation moved this from 🔖 To do to 🍹 Done Jun 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@reckart reckart

Labels
🚀 Performance Performance / scalability ⚙️ Refactoring
Projects
Kanban
  
🍹 Done
Milestone
24.0
Development

No branches or pull requests
1 participant
@reckart