Comment thread objects (MBDiscussion, MBThread and root MBMessage) are created during asset creation, but that it is not necessary as CommentManager also handle it

• Type: Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: 6.2.10 EE GA1, 6.2.X EE, 7.0.0 CE GA1, 7.0.1 CE GA2
• Fix Version/s: 7.0.0 DXP FP3, 7.0.0 DXP SP1, Master
• Component/s: Collaboration, Comments
• Labels:
  • QA-CT
  • backend_test
  • liferay-digital-enterprise-70-sp1
  • liferay-fixpack-de-3-7010
  • post-upgrade-fix
• Branch Version/s:
  7.0.x
• Backported to Branch:
  Committed
• Story Points: 1
• Fix Priority: 3
• Git Pull Request: https://github.com/brianchandotcom/liferay-portal/pull/41933

In LPS-2864 was added some logic in order to create all comment thread structure during asset creation (MBDiscussion, MBThread and root MBMessage objects) but that it is not necessary as CommentManager also handle it during comment creation.

After some testing we have detected that all MBThread entries causes an overhead in the system, for example:

• During staging reindex operation, a 30% of improvement is obtained deactivating deleting all those objects, and comment functionality is not affected.
• During full reindex operation, some customers have reported that 25%-50% of the time is spent during MBThread reindex

In order to improve throughput, we should:

1. Remove all comment creation logic from assets creation (JournalArticleLocalServiceImpl.addArticle, BlogsEntryLocalServiceImpl.addEntry, Layout, ....)
2. Clean all unused objects during upgrade to 7.0