fixup! Exclude JsonIgnore PipelineOptions from DisplayData

apache · May 10, 2016 · d82d1a5 · d82d1a5
1 parent 8fe5624
commit d82d1a5
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/sdks/java/core/src/main/java/org/apache/beam/sdk/options/ProxyInvocationHandler.java b/sdks/java/core/src/main/java/org/apache/beam/sdk/options/ProxyInvocationHandler.java
@@ -282,6 +282,8 @@ private void populateDisplayData(DisplayData.Builder builder) {
 
       for (PipelineOptionSpec optionSpec : specs) {
         if (!optionSpec.shouldSerialize()) {
+          // Options that are excluded for serialization (i.e. those with @JsonIgnore) are also
+          // excluded from display data. These options are generally not useful for display.
           continue;
         }