Paul Rogers created DRILL-5103:
----------------------------------
Summary: External Sort pop shadows variable "initialAllocation"
from AbstractBase
Key: DRILL-5103
URL: https://issues.apache.org/jira/browse/DRILL-5103
Project: Apache Drill
Issue Type: Bug
Affects Versions: 1.8.0
Reporter: Paul Rogers
Assignee: Paul Rogers
The Physical Operator (pop) definition for External sort contains the following:
{code}
public class ExternalSort extends Sort {
private long initialAllocation = 20000000;
{code}
But,
{code}
public abstract class AbstractBase implements PhysicalOperator{
protected long initialAllocation = 1000000L;
{code}
The result is that the variable in {{ExternalSort}} shadows the one in
{{AbstractBase}}. Jackson deserialization sets the one in {{AbstractBase}}, but
the method {{getInitialAllocation()}} returns the shadowed one in
{{ExternalSort}}.
Remove the shadowed {{initialAllocation}} from {{ExternalSort}} and move the
initializer into the constructor.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
