[jira] [Created] (DRILL-5103) External Sort pop shadows variable "initialAllocation" from AbstractBase

["Paul Rogers (JIRA)" <ji...@apache.org>]

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)