Liam Miller-Cushon created MAPREDUCE-7364:
---------------------------------------------

             Summary: Fragile reference to private synthetic this$ field
                 Key: MAPREDUCE-7364
                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-7364
             Project: Hadoop Map/Reduce
          Issue Type: Bug
          Components: client
            Reporter: Liam Miller-Cushon


The following code is using reflection to access a field with a name starting 
with `this$`:

[https://github.com/apache/hadoop/blob/35eff545560d9275a03d3d2dcb019d7cdfc39e3e/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/ContextFactory.java#L132]

The OpenJDK javac generates private synthetic fields with names starting with 
`this$` as an implementation detail of inner classes. In the future that 
implementation detail may be changing, and the `this$` field will no longer be 
generated for all inner classes. For more information about the proposed 
change, see: https://bugs.openjdk.java.net/browse/JDK-8271717

Please consider alternatives to accessing the private synthetic `this$` field 
to ensure this code continues to work after the change. For example, consider 
passing an explicit copy of the enclosing instance to code that needs access to 
it, or adding an explicit getter to the inner class.

For example, given:

{code:java}
class Outer {
  int x;
  class Inner1 {
    int f() {
      return x;
    }
  }
  class Inner2 {
    void g() {
      System.err.println("hello");
    }
  }
}
{code}

Currently `Inner1` and `Inner2` both have a synthetic field named `this$0` that 
stores a reference to `Outer`.

In the future the implementation detail might be changing to omit the field 
from classes that don't reference the enclosing instance. So in the example, 
`Inner1` would still have the synthetic field because it accesses the field `x` 
in the enclosing instance `Outer`. However `Inner2` would no longer have a 
synthetic field, because it doesn't access any state from its enclosing 
instance.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

Reply via email to