[ https://issues.apache.org/jira/browse/TEZ-4106?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17029333#comment-17029333 ]
Jonathan Turner Eagles commented on TEZ-4106: --------------------------------------------- [~ahussein], Couple of minor comments I should have caught before. The javadoc for TEZ_AM_SPECULATOR_CLASS and TEZ_AM_SPECULATOR_ESTIMATOR_CLASS should be change to differentiate them. scanForSpeculationsForTesting uses both LOG.info and System.out.println as well as a comment saying // debug. Makes me think they were intended to be removed. > Add Exponential Smooth RuntimeEstimator to the speculator > --------------------------------------------------------- > > Key: TEZ-4106 > URL: https://issues.apache.org/jira/browse/TEZ-4106 > Project: Apache Tez > Issue Type: Improvement > Reporter: Ahmed Hussein > Assignee: Ahmed Hussein > Priority: Major > Attachments: TEZ-4106.001.patch, TEZ-4106.002.patch, > TEZ-4106.003.patch, TEZ-4106.004.patch, TEZ-4106.005.patch > > > Tez speculator implements start-end runtime estimator. Similar to > [MAPREDUCE-7208|https://issues.apache.org/jira/browse/MAPREDUCE-7208], we > need to implement an adaptive estimator based on smooth Exponential -- This message was sent by Atlassian Jira (v8.3.4#803005)