drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Paul Rogers (JIRA)" <j...@apache.org>
Subject [jira] [Created] (DRILL-5103) External Sort pop shadows variable "initialAllocation" from AbstractBase
Date Sun, 04 Dec 2016 07:39:59 GMT
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:

public class ExternalSort extends Sort {
  private long initialAllocation = 20000000;


public abstract class AbstractBase implements PhysicalOperator{
  protected long initialAllocation = 1000000L;

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

View raw message