[ 
https://issues.apache.org/jira/browse/LENS-1381?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15857558#comment-15857558
 ] 

Amareshwari Sriramadasu commented on LENS-1381:
-----------------------------------------------

Comments on the code in branch lens-1381 :

# TODO statements in FactPartition class does not look relevant. Can be removed.
# Many TODO statements in various classes should be removed or acted upon.
# Javadoc on Candidate interface needs update.
# I see lots of code throwing 'throw new LensException("Not a 
StorageCandidate!!");' - we need to understand if  it will be hit in any code 
flow.
# Why is storateTable renamed to storageName in CandidateDim?
# '  String getStorageName();' in CandidateTable interface should be renamed to 
getStorageTableName().
# I feel, with CandidateTableResolver checkForQueriedColumns should happen 
before populating storage candidates
# Populating of factDimMap in CubeQueryContext.toHQL() might need changes
# DenormalizationResolver and ExpressionResolver needs merge with master
# What does position in QueriedPhraseContext signify? The queried phrase can be 
in any clause - select, where or group by and etc. What does the position say 
here?


> Support Fact to Fact Union
> --------------------------
>
>                 Key: LENS-1381
>                 URL: https://issues.apache.org/jira/browse/LENS-1381
>             Project: Apache Lens
>          Issue Type: New Feature
>            Reporter: Puneet Gupta
>            Assignee: Puneet Gupta
>
> Currently Lens supports Union-ing data across different storages in a single 
> Fact. With this JIRA Lens server will be able to Union Data Across Facts too.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to