>From Ian Maxon <[email protected]>: Ian Maxon has uploaded a new patch set (#2). ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17962 )
Change subject: [ASTERIXDB-3313][COMP] Defuse profile when not profiling ...................................................................... [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 --- M hyracks-fullstack/algebricks/algebricks-compiler/src/main/java/org/apache/hyracks/algebricks/compiler/api/HeuristicCompilerFactoryBuilder.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/jobgen/impl/JobBuilder.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/jobgen/impl/PlanCompiler.java M hyracks-fullstack/algebricks/algebricks-compiler/src/main/java/org/apache/hyracks/algebricks/compiler/api/ICompiler.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java M hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/meta/PipelineAssembler.java 7 files changed, 71 insertions(+), 8 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/62/17962/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17962 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: trinity Gerrit-Change-Id: Ia9a84eedbcb33c29f03155a8605bb82af372f1f3 Gerrit-Change-Number: 17962 Gerrit-PatchSet: 2 Gerrit-Owner: Ian Maxon <[email protected]> Gerrit-CC: Jenkins <[email protected]> Gerrit-MessageType: newpatchset
