[SENTRY-1909] Improvements for memory usage when full path snapshot is sent from Sentry to NN

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.0
• Component/s: Sentry
• Assignee: Alex Kolbasov
• Reporter: Alex Kolbasov
• Created: 31/Aug/17 00:52
• Updated: 20/Nov/17 21:16
• Resolved: 02/Sep/17 01:38

Description

While looking at SENTRY-1907 I noticed another thing. sentryStore.retrieveFullPathsImage() uses Map<String, Set<String>> as a snapshot representation. This isn't needed since Map<String, Collection<String>> is sufficient and we can represent it using ArrayList. This is much more efficient representation.

Issue Links

• is related to: SENTRY-1912 Directly construct Thrift message from DB when full update is sent to NN
• is related to: SENTRY-1915 Sentry is doing a lot of work to convert list of paths to HMSPaths structure
• links to: Code review