[
https://issues.apache.org/jira/browse/ASTERIXDB-3313?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17787513#comment-17787513
]
ASF subversion and git services commented on ASTERIXDB-3313:
------------------------------------------------------------
Commit 6c1e847f1902e1ba710134b0ab15fd7daf76082d in asterixdb's branch
refs/heads/master from Ian Maxon
[ https://gitbox.apache.org/repos/asf?p=asterixdb.git;h=6c1e847f19 ]
[ASTERIXDB-3313][COMP] Defuse profile when not profiling
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
The logical2physical map is time consuming to produce, so it should
not be computed unless it's needed for a profile. Likewise keeping
the last plan in memory is wasteful and useless unless a profile is
needed. Lastly in the pipeline assembler, look at the profile flag
first instead of the instanceof check to save time.
Change-Id: Ia9a84eedbcb33c29f03155a8605bb82af372f1f3
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17962
Integration-Tests: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>
Reviewed-by: Ian Maxon <[email protected]>
Reviewed-by: Ali Alsuliman <[email protected]>
> All profiling-related methods should be disabled when not profiling
> -------------------------------------------------------------------
>
> Key: ASTERIXDB-3313
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-3313
> Project: Apache AsterixDB
> Issue Type: Bug
> Components: COMP - Compiler, RT - Runtime
> Reporter: Ian Maxon
> Assignee: Ian Maxon
> Priority: Major
> Labels: triaged
> Fix For: 0.9.9
>
>
> There are some costly maps the compiler needs to keep track of to tie the
> runtime profile back to the plan. Currently these are computed whether or not
> the profile is going to be present or not.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
