[ https://issues.apache.org/jira/browse/HIVE-12312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14992465#comment-14992465 ]
Hive QA commented on HIVE-12312: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12770815/HIVE-12312.1.patch {color:red}ERROR:{color} -1 due to build exiting with an error Test results: http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/5935/testReport Console output: http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/5935/console Test logs: http://ec2-174-129-184-35.compute-1.amazonaws.com/logs/PreCommit-HIVE-TRUNK-Build-5935/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Tests exited with: NonZeroExitCodeException Command 'bash /data/hive-ptest/working/scratch/source-prep.sh' failed with exit status 1 and output '+ [[ -n /usr/java/jdk1.7.0_45-cloudera ]] + export JAVA_HOME=/usr/java/jdk1.7.0_45-cloudera + JAVA_HOME=/usr/java/jdk1.7.0_45-cloudera + export PATH=/usr/java/jdk1.7.0_45-cloudera/bin/:/usr/java/jdk1.7.0_45-cloudera/bin:/usr/local/apache-maven-3.0.5/bin:/usr/local/apache-maven-3.0.5/bin:/usr/java/jdk1.7.0_45-cloudera/bin:/usr/local/apache-ant-1.9.1/bin:/usr/local/bin:/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/sbin:/home/hiveptest/bin + PATH=/usr/java/jdk1.7.0_45-cloudera/bin/:/usr/java/jdk1.7.0_45-cloudera/bin:/usr/local/apache-maven-3.0.5/bin:/usr/local/apache-maven-3.0.5/bin:/usr/java/jdk1.7.0_45-cloudera/bin:/usr/local/apache-ant-1.9.1/bin:/usr/local/bin:/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/sbin:/home/hiveptest/bin + export 'ANT_OPTS=-Xmx1g -XX:MaxPermSize=256m ' + ANT_OPTS='-Xmx1g -XX:MaxPermSize=256m ' + export 'M2_OPTS=-Xmx1g -XX:MaxPermSize=256m -Dhttp.proxyHost=localhost -Dhttp.proxyPort=3128' + M2_OPTS='-Xmx1g -XX:MaxPermSize=256m -Dhttp.proxyHost=localhost -Dhttp.proxyPort=3128' + cd /data/hive-ptest/working/ + tee /data/hive-ptest/logs/PreCommit-HIVE-TRUNK-Build-5935/source-prep.txt + [[ false == \t\r\u\e ]] + mkdir -p maven ivy + [[ git = \s\v\n ]] + [[ git = \g\i\t ]] + [[ -z master ]] + [[ -d apache-github-source-source ]] + [[ ! -d apache-github-source-source/.git ]] + [[ ! -d apache-github-source-source ]] + cd apache-github-source-source + git fetch origin >From https://github.com/apache/hive 33e818a..d4abd51 branch-1 -> origin/branch-1 a46729b..d33ddef master -> origin/master + git reset --hard HEAD HEAD is now at a46729b HIVE-12156: expanding view doesn't quote reserved keyword (Pengcheng Xiong, reviewed by Laljo John Pullokkaran) + git clean -f -d Removing ql/src/test/queries/clientnegative/set_hiveconf_internal_variable0.q Removing ql/src/test/queries/clientnegative/set_hiveconf_internal_variable1.q Removing ql/src/test/results/clientnegative/set_hiveconf_internal_variable0.q.out Removing ql/src/test/results/clientnegative/set_hiveconf_internal_variable1.q.out + git checkout master Already on 'master' Your branch is behind 'origin/master' by 1 commit, and can be fast-forwarded. + git reset --hard origin/master HEAD is now at d33ddef HIVE-12340 : ExecDriver.execute() unnecessarily sets METASTOREPWD to HIVE (Hari Subramaniyan, reviewed by Sushanth Sowmyan) + git merge --ff-only origin/master Already up-to-date. + git gc + patchCommandPath=/data/hive-ptest/working/scratch/smart-apply-patch.sh + patchFilePath=/data/hive-ptest/working/scratch/build.patch + [[ -f /data/hive-ptest/working/scratch/build.patch ]] + chmod +x /data/hive-ptest/working/scratch/smart-apply-patch.sh + /data/hive-ptest/working/scratch/smart-apply-patch.sh /data/hive-ptest/working/scratch/build.patch The patch does not appear to apply with p0, p1, or p2 + exit 1 ' {noformat} This message is automatically generated. ATTACHMENT ID: 12770815 - PreCommit-HIVE-TRUNK-Build > Excessive logging in PPD code > ----------------------------- > > Key: HIVE-12312 > URL: https://issues.apache.org/jira/browse/HIVE-12312 > Project: Hive > Issue Type: Bug > Components: Hive > Affects Versions: 1.2.1 > Reporter: Carter Shanklin > Priority: Minor > Attachments: HIVE-12312.1.patch > > > One of my very complex queries takes about 14 minutes to compile with PPD on. > Profiling it I saw a lot of time spent in this stack which is called many > many thousands of times. > {code} > java.lang.Throwable.getStackTraceElement(-2) > java.lang.Throwable.getOurStackTrace(827) > java.lang.Throwable.getStackTrace(816) > sun.reflect.GeneratedMethodAccessor5.invoke(-1) > sun.reflect.DelegatingMethodAccessorImpl.invoke(43) > java.lang.reflect.Method.invoke(497) > org.apache.log4j.spi.LocationInfo.<init>(139) > org.apache.log4j.spi.LoggingEvent.getLocationInformation(253) > org.apache.log4j.helpers.PatternParser$LocationPatternConverter.convert(500) > org.apache.log4j.helpers.PatternConverter.format(65) > org.apache.log4j.PatternLayout.format(506) > org.apache.log4j.WriterAppender.subAppend(310) > org.apache.log4j.DailyRollingFileAppender.subAppend(369) > org.apache.log4j.WriterAppender.append(162) > org.apache.log4j.AppenderSkeleton.doAppend(251) > org.apache.log4j.helpers.AppenderAttachableImpl.appendLoopOnAppenders(66) > org.apache.log4j.Category.callAppenders(206) > org.apache.log4j.Category.forcedLog(391) > org.apache.log4j.Category.log(856) > org.apache.commons.logging.impl.Log4JLogger.info(176) > org.apache.hadoop.hive.ql.ppd.OpProcFactory$DefaultPPD.logExpr(707) > org.apache.hadoop.hive.ql.ppd.OpProcFactory$DefaultPPD.mergeWithChildrenPred(752) > org.apache.hadoop.hive.ql.ppd.OpProcFactory$FilterPPD.process(437) > {code} > logExpr is set to log at INFO level, but I think DEBUG is more appropriate. > When I set log level to debug I see > 20% speedup in compile time: > Before: > {code} > real 14m47.972s > user 15m25.609s > sys 0m20.282s > {code} > After: > {code} > real 11m30.946s > user 12m10.870s > sys 0m7.320s > {code} > It looks like there's a lot of stuff in the PPD code that could be optimized, > when I turn PPD off the query compiles in 2m 30s. But this seems like an easy > and low risk win. -- This message was sent by Atlassian JIRA (v6.3.4#6332)