feature request #5752
Reduce TaxEditor size
100%
Description
There are many places where the Editor code needs to be cleaned up in terms of real dependencies and code which links to dependencies but is not necessary.
Most important issue is to keep only 1 instance of cdmlib-remote-webapp. Currently there is 1 instance in taxeditor.cdmlib and 1 in taxeditor.remote-webapp.
Also important to reduce the dependencies in taxeditor.cdmlib. With using only remoting from now on, we do not need most of the cdmlib-dependencies anymore. This will reduce the current size of libraries of 120MB tremendously.
Further tasks are to clean up all poms and Manifest.MFs (see also #5530) etc.
Idea from #967: Split the cdmlib plugin into two plugins: One for cdmlib dependencies and one for taxeditor-only dependencies
Related issues
Associated revisions
ref #5752 cleanup dependencies in taxeditor.cdmlib
ref #5752 cleanup dependencies in taxeditor.cdmlib Part 2
ref #5752 cleanup dependencies in taxeditor.cdmlib Part 3
ref #5752 cleanup dependencies in taxeditor.cdmlib Part 4
ref #5752 revert last 3 "cleanup dependencies in taxeditor.cdmlib"
ref #5752 wrap specific exception to avoid dependency on lucene library
ref #5752 Fix dependencies for integration tests
ref #5752 Remove jars from lib folder
ref #5752 wrap lucene exception for name and occurrence service
ref #5752 Adapt implementations to new wrapper for lucene exceptions
ref #5752 Thin out feature.platform dependencies
- also gets rid of "Run" menu
ref #5752 Add missing dependency
ref #5752 Re-add dependencies needed for creating a new data model
remove xstream and dependencies xpp3(_min) and xmlpull from taxeditor.cdmlib plugin
remove mysql-connector-java from taxeditor-cdmlib
update mysql-connector to 8.0.20 in taxeditor-webapp
ref #5752 remove cdmlib-remote-webapp.war from taxeditor.cdmlib
ref #5752 remove cdmlib-remote-webapp.war from taxeditor.cdmlib (cont.)
ref #5752 remove cdmlib-remote-webapp.war from taxeditor.cdmlib (cont.)
ref #5752 remove easymock-2.3, easymockclassextension-2.3, h2-1.4.190, h2mig_pagestore_addon, hsqldb-2.3.3, junit-benchmarks-0.7.2, ognl-2.6.9, yip-controller-api-redist from taxeditor.cdmlib
ref #5752 add h2-1.4.200 to taxeditor.test
ref #5752 use taxeditor.webapp cdmlib-remote-webapp.war for taxeditor tests
ref #5752 remove javax.servlet-api and spring-webmvc
ref #5752 remove spring-modules-cache and dependeny concurrent-1.3.4
ref #5752 remove spring-security-web
ref #5752 remove spring-security-oauth2 including jackson-mapper-asl and jackson-core-asl
ref #5752 remove dozer-5.3.0
ref #5752 remove hibernate-cglib-repack-2.1_3, hibernate-ehcache-5.0.7.Final
ref #5752 remove xom and springfox-swagger-common-2.3.1 and springfox-swagger2-2.3.1
ref #5752 reattache H2 to TaxEditor dependencies
History
#1 Updated by Andreas Müller almost 5 years ago
- Assignee changed from p.plitzner - to Andreas Müller
- Target version changed from Unassigned CDM tickets to Release 4.1
- Priority changed from New to Highest
#2 Updated by Andreas Müller almost 5 years ago
- Target version changed from Release 4.1 to Release 4.2
#3 Updated by Andreas Müller almost 5 years ago
- Target version changed from Release 4.2 to Release 4.3
#4 Updated by Andreas Müller over 4 years ago
- Target version changed from Release 4.3 to Release 4.4
#5 Updated by Andreas Müller over 4 years ago
- Target version changed from Release 4.4 to Release 4.5
#6 Updated by Andreas Müller about 4 years ago
- Target version changed from Release 4.5 to Release 4.6
#7 Updated by Patrick Plitzner about 4 years ago
- Description updated (diff)
- Status changed from New to In Progress
- Assignee changed from Andreas Müller to Patrick Plitzner
- Target version changed from Release 4.6 to Release 4.7
#8 Updated by Patrick Plitzner almost 4 years ago
With the commit taxeditor|d4e83e37febf32601f065390770060398ec63a80. I re-added the dependencies needed for creating a fresh, new data model for an empty data source.
These dependencies can be removed when we moved this functionality completely to the server side
#9 Updated by Andreas Müller almost 4 years ago
- Target version changed from Release 4.7 to Release 4.8
- Private changed from Yes to No
#10 Updated by Andreas Müller almost 4 years ago
- Target version changed from Release 4.8 to Release 4.9
#11 Updated by Andreas Müller over 3 years ago
- Target version changed from Release 4.9 to Release 4.10
#12 Updated by Andreas Müller over 3 years ago
- Target version changed from Release 4.10 to Release 4.12
#13 Updated by Andreas Müller over 3 years ago
- Target version changed from Release 4.12 to Release 4.13
#14 Updated by Andreas Müller about 3 years ago
- Target version changed from Release 4.13 to Release 4.14
#15 Updated by Andreas Müller about 3 years ago
- Target version changed from Release 4.14 to Release 5.0
#16 Updated by Andreas Müller about 3 years ago
- Target version changed from Release 5.0 to Release 5.1
#17 Updated by Patrick Plitzner about 3 years ago
- Related to bug #7306: No initial default classification is created when creating a new data source added
#18 Updated by Andreas Müller almost 3 years ago
- Target version changed from Release 5.1 to Release 5.2
#19 Updated by Andreas Müller over 2 years ago
- Target version changed from Release 5.2 to Release 5.3
#21 Updated by Patrick Plitzner over 2 years ago
- Description updated (diff)
#22 Updated by Patrick Plitzner over 2 years ago
- Target version changed from Release 5.3 to Release 5.4
#23 Updated by Patrick Plitzner over 2 years ago
- Target version changed from Release 5.4 to Release 5.5
#24 Updated by Patrick Plitzner over 2 years ago
- Target version changed from Release 5.5 to Release 5.6
#25 Updated by Andreas Müller about 2 years ago
- Target version changed from Release 5.6 to Reviewed Next Major Release
#26 Updated by Andreas Müller over 1 year ago
- Assignee changed from Patrick Plitzner to Katja Luther
#27 Updated by Andreas Müller 9 months ago
- Related to task #9119: Remove schema create and schema update functionality from datsource view added
#28 Updated by Andreas Müller 9 months ago
- Target version changed from Reviewed Next Major Release to Release 5.18
cdmlib-remote-webapp is now removed from taxeditor.cdmlib which reduces the size of TaxEditor about 115 MB
#29 Updated by Andreas Müller 9 months ago
- Status changed from In Progress to Resolved
- Assignee changed from Katja Luther to Andreas Müller
Many improvements have been done in this ticket. For further improvements a follow up ticket exists: #9154
Open issue is still to reattache H2 and PostGreSQL drivers as they are needed for database view edit dialog for testing the connection or find a better solution for testing. Datasource view could be moved to taxeditor.webapp as it is only relevant together with it.
#30 Updated by Andreas Müller 9 months ago
- Subject changed from Clean up Editor dependencies to Reduce TaxEditor size
#31 Updated by Andreas Müller 9 months ago
- Copied to bug #9145: Add sources to E+M IPNI distribution data added
#32 Updated by Andreas Müller 9 months ago
- % Done changed from 0 to 80
#33 Updated by Andreas Müller 9 months ago
- Copied to feature request #9154: Further reduce TaxEditor size by cleaning up dependencies and making taxeditor.webapp plugin optional added
#34 Updated by Andreas Müller 9 months ago
- Copied to deleted (bug #9145: Add sources to E+M IPNI distribution data)
#35 Updated by Andreas Müller 8 months ago
- Description updated (diff)
#36 Updated by Andreas Müller 8 months ago
- Related to task #9189: Move datasource view to taxeditor.local added
#37 Updated by Andreas Müller 8 months ago
- Assignee changed from Andreas Müller to Katja Luther
Can you please review if you think all up-to-now changes (removing of dependencies etc) are ok. I think you do not need to review Patricks changes from 3 years ago as they are in production mode for long time, now.
There is a follow-up ticket for further size reductions.
#38 Updated by Katja Luther 8 months ago
- Status changed from Resolved to Closed
- Assignee changed from Katja Luther to Andreas Müller
- % Done changed from 80 to 100
I checked all removed dependencies and they are not used anyhwere and I did not get any problems while working with the editor, so I close this tickeet.
#39 Updated by Andreas Kohlbecker 8 months ago
- Status changed from Closed to Feedback
Can you please add a summary to the ticked description on the amount of the size reduction?
This could be valuable for later.
#40 Updated by Andreas Müller 8 months ago
- Status changed from Feedback to Closed
The size reduction was about 120MB. Most of it (110MB) related to the removed remote-webapp.war file in taxeditor.cdmlib. The remaining related to some dependencies that removed in taxeditor.cdmlib.
#41 Updated by Andreas Müller 8 months ago
- Target version changed from Release 5.18 to Release 5.17