[
https://issues.apache.org/jira/browse/TAJO-1430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14446580#comment-14446580
]
ASF GitHub Bot commented on TAJO-1430:
--------------------------------------
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/tajo/pull/442#discussion_r27782120
--- Diff: tajo-core/src/main/java/org/apache/tajo/session/Session.java ---
@@ -121,6 +124,10 @@ public synchronized String getCurrentDatabase() {
return currentDatabase;
}
+ public synchronized void setQueryCache(LoadingCache<String, Expr> cache)
{ this.cache = cache; }
--- End diff --
Currently, in this patch, I try to minimize dependency change in order to
keep Tajo's good modularity. So, I put only Expr dependency into Session class.
(It's inevitable.) If we put the cache initialization there, we need "import
org.apache.tajo.engine.parser.SQLAnalyzer" there. I am not sure that Tajo
allows this kind of dependency. If you think this is okay, I prefer to move
that into Session. I need your guide. :)
> Improve SQLAnalyzer by session-based parsing-result caching
> -----------------------------------------------------------
>
> Key: TAJO-1430
> URL: https://issues.apache.org/jira/browse/TAJO-1430
> Project: Tajo
> Issue Type: New Feature
> Components: parser
> Affects Versions: 0.10.0
> Reporter: Dongjoon Hyun
> Assignee: Dongjoon Hyun
> Fix For: 0.10.1
>
> Attachments: TAJO-1430.patch, long_2times.sql, wide_table.sql
>
>
> There are wide tables with many many columns. Moveover, BI tools generate
> very complex queries whose size is several MB. Although Tajo executes those
> queries very fast in a few seconds, the total time of UX is slow.
> To become a fastest Hadoop DW, we need this following feature.
> {code:sql}
> tsql -f long_2times.sql
> ...
> (0 rows, 30.641 sec, 0 B selected)
> ...
> (0 rows, 1.707 sec, 0 B selected)
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
