GitHub user navis opened a pull request:

    https://github.com/apache/tajo/pull/527

    TAJO-1505 Transferred plan should not include supplementary of FunctionDesc

    Considered to use node visitor, but seemed better not to modify internal 
shape of the plan. All modifications are just for doing,
    
    ```java
    @Override
    public FunctionSupplementProto getProto(SerializeOption option) {
      FunctionSupplementProto.Builder builder = 
FunctionSupplementProto.newBuilder();
      if (shortDescription != null && option != SerializeOption.INTERNAL) {
        builder.setShortDescription(shortDescription);
      }
      if (detail != null && option != SerializeOption.INTERNAL) {
        builder.setDetail(detail);
      }
      if (example != null && option != SerializeOption.INTERNAL) {
        builder.setExample(example);
      }
      return builder.build();
    }
    ```

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/navis/tajo TAJO-1505

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tajo/pull/527.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #527
    
----
commit 91fd08e218662486f10303b1729dc20d4d505c6b
Author: navis.ryu <[email protected]>
Date:   2015-04-08T04:45:32Z

    TAJO-1505 Transferred plan should not include supplementary of FunctionDesc

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to