Project

General

Profile

Actions

feature request #5752

closed

Reduce TaxEditor size

Added by Andreas Müller over 6 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Highest
Category:
taxeditor
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
normal

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

Related to EDIT - bug #7306: No initial default classification is created when creating a new data sourceNewKatja Luther

Actions
Related to EDIT - task #9119: Remove schema create and schema update functionality from datsource viewClosedKatja Luther

Actions
Related to EDIT - task #9189: Move datasource view to taxeditor.localClosedKatja Luther

Actions
Copied to EDIT - feature request #9154: Further reduce TaxEditor size by cleaning up dependencies and making taxeditor.webapp plugin optionalNewKatja Luther

Actions
Actions #1

Updated by Andreas Müller over 6 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
Actions #2

Updated by Andreas Müller over 6 years ago

  • Target version changed from Release 4.1 to Release 4.2
Actions #3

Updated by Andreas Müller over 6 years ago

  • Target version changed from Release 4.2 to Release 4.3
Actions #4

Updated by Andreas Müller about 6 years ago

  • Target version changed from Release 4.3 to Release 4.4
Actions #5

Updated by Andreas Müller about 6 years ago

  • Target version changed from Release 4.4 to Release 4.5
Actions #6

Updated by Andreas Müller almost 6 years ago

  • Target version changed from Release 4.5 to Release 4.6
Actions #7

Updated by Patrick Plitzner over 5 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
Actions #8

Updated by Patrick Plitzner over 5 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

Actions #9

Updated by Andreas Müller over 5 years ago

  • Target version changed from Release 4.7 to Release 4.8
  • Private changed from Yes to No
Actions #10

Updated by Andreas Müller over 5 years ago

  • Target version changed from Release 4.8 to Release 4.9
Actions #11

Updated by Andreas Müller over 5 years ago

  • Target version changed from Release 4.9 to Release 4.10
Actions #12

Updated by Andreas Müller about 5 years ago

  • Target version changed from Release 4.10 to Release 4.12
Actions #13

Updated by Andreas Müller about 5 years ago

  • Target version changed from Release 4.12 to Release 4.13
Actions #14

Updated by Andreas Müller almost 5 years ago

  • Target version changed from Release 4.13 to Release 4.14
Actions #15

Updated by Andreas Müller almost 5 years ago

  • Target version changed from Release 4.14 to Release 5.0
Actions #16

Updated by Andreas Müller almost 5 years ago

  • Target version changed from Release 5.0 to Release 5.1
Actions #17

Updated by Patrick Plitzner over 4 years ago

  • Related to bug #7306: No initial default classification is created when creating a new data source added
Actions #18

Updated by Andreas Müller over 4 years ago

  • Target version changed from Release 5.1 to Release 5.2
Actions #19

Updated by Andreas Müller over 4 years ago

  • Target version changed from Release 5.2 to Release 5.3
Actions #21

Updated by Patrick Plitzner over 4 years ago

  • Description updated (diff)
Actions #22

Updated by Patrick Plitzner about 4 years ago

  • Target version changed from Release 5.3 to Release 5.4
Actions #23

Updated by Patrick Plitzner about 4 years ago

  • Target version changed from Release 5.4 to Release 5.5
Actions #24

Updated by Patrick Plitzner about 4 years ago

  • Target version changed from Release 5.5 to Release 5.6
Actions #25

Updated by Andreas Müller almost 4 years ago

  • Target version changed from Release 5.6 to Reviewed Next Major Release
Actions #26

Updated by Andreas Müller about 3 years ago

  • Assignee changed from Patrick Plitzner to Katja Luther
Actions #27

Updated by Andreas Müller over 2 years ago

  • Related to task #9119: Remove schema create and schema update functionality from datsource view added
Actions #28

Updated by Andreas Müller over 2 years 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

Actions #29

Updated by Andreas Müller over 2 years 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.

Actions #30

Updated by Andreas Müller over 2 years ago

  • Subject changed from Clean up Editor dependencies to Reduce TaxEditor size
Actions #31

Updated by Andreas Müller over 2 years ago

  • Copied to bug #9145: Add sources to E+M IPNI distribution data added
Actions #32

Updated by Andreas Müller over 2 years ago

  • % Done changed from 0 to 80
Actions #33

Updated by Andreas Müller over 2 years ago

  • Copied to feature request #9154: Further reduce TaxEditor size by cleaning up dependencies and making taxeditor.webapp plugin optional added
Actions #34

Updated by Andreas Müller over 2 years ago

  • Copied to deleted (bug #9145: Add sources to E+M IPNI distribution data)
Actions #35

Updated by Andreas Müller over 2 years ago

  • Description updated (diff)
Actions #36

Updated by Andreas Müller over 2 years ago

  • Related to task #9189: Move datasource view to taxeditor.local added
Actions #37

Updated by Andreas Müller over 2 years 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.

Actions #38

Updated by Katja Luther over 2 years 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.

Actions #39

Updated by Andreas Kohlbecker over 2 years 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.

Actions #40

Updated by Andreas Müller over 2 years 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.

Actions #41

Updated by Andreas Müller over 2 years ago

  • Target version changed from Release 5.18 to Release 5.17
Actions

Also available in: Atom PDF