[
https://issues.apache.org/jira/browse/LENS-14?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14216401#comment-14216401
]
Rajat Khandelwal commented on LENS-14:
--------------------------------------
The function resolveDimStorageTablesAndPartitions needs to be split into two
functions: resolveDimStorageTables and resolveDimStoragePartitions. Then
StorageTableResolver will be split into two classes. StorageTableResolver and
StoragePartitionResolver. In the list of resolvers, LightestFactResolver will
be added either in between of the two mentioned earlier or after both depending
on whether it's turned on in config.
I'll implement it if the approach sounds fine.
> Lightest fact first config wont work with more than one storage
> ---------------------------------------------------------------
>
> Key: LENS-14
> URL: https://issues.apache.org/jira/browse/LENS-14
> Project: Apache Lens
> Issue Type: Bug
> Components: cube
> Reporter: Amareshwari Sriramadasu
> Fix For: 2.0
>
>
> With lightest fact resolver first config enabled, it would pick only one
> lightest fact to be lightest. And storage table resolver might skip it if
> current driver cannot work on the storage chosen.
> The problem is StorageTableResolver resolves supported storages, final
> storage tables and also checks for partitions available in tables. The
> solution is to split StorageTableResolver into two resolvers sothat we first
> apply storage table resolver to find supported storages and storage tables.
> If lightest fact first config enabled, then apply LightestFactResolver and
> then apply storage partition resolver, which checks for partition existence
> on the fact storage tables.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
