This is an automated email from the ASF dual-hosted git repository. liujun pushed a commit to branch dev-metadata in repository https://gitbox.apache.org/repos/asf/incubator-dubbo.git
commit 453e323b4b955505b9c3ef10340f55ab8a3ca4a0 Author: ken.lj <[email protected]> AuthorDate: Thu Nov 15 17:58:54 2018 +0800 ignore parameter(exclude=true). --- .../src/main/java/org/apache/dubbo/config/AbstractConfig.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractConfig.java b/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractConfig.java index 130aa03..0be4c41 100644 --- a/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractConfig.java +++ b/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractConfig.java @@ -459,7 +459,9 @@ public abstract class AbstractConfig implements Serializable { } else { key = prop; } - if (method.getReturnType() == Object.class || parameter != null && parameter.excluded()) { + // treat url and configuration differently, the value should always present in configuration though it may not need to present in url. + //if (method.getReturnType() == Object.class || parameter != null && parameter.excluded()) { + if (method.getReturnType() == Object.class) { metaData.put(key, null); continue; }
