Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
doujutun3207
flink
提交
63ef8e86
F
flink
项目概览
doujutun3207
/
flink
与 Fork 源项目一致
从无法访问的项目Fork
通知
24
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
F
flink
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
63ef8e86
编写于
12月 16, 2014
作者:
R
Robert Metzger
提交者:
mbalassi
12月 17, 2014
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
[FLINK-1333] Fixed getter/setter recognition for POJOs
This closes #271
上级
a70aa67a
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
33 addition
and
4 deletion
+33
-4
flink-java/src/main/java/org/apache/flink/api/java/typeutils/TypeExtractor.java
...va/org/apache/flink/api/java/typeutils/TypeExtractor.java
+4
-4
flink-java/src/test/java/org/apache/flink/api/java/type/extractor/PojoTypeExtractionTest.java
...flink/api/java/type/extractor/PojoTypeExtractionTest.java
+29
-0
未找到文件。
flink-java/src/main/java/org/apache/flink/api/java/typeutils/TypeExtractor.java
浏览文件 @
63ef8e86
...
...
@@ -989,12 +989,12 @@ public class TypeExtractor {
}
for
(
Method
m
:
clazz
.
getMethods
())
{
// check for getter
if
(
// The name should be "get<FieldName>" or "<fieldName>" (for scala).
(
m
.
getName
().
toLowerCase
().
equals
(
"get"
+
fieldNameLow
)
||
m
.
getName
().
toLowerCase
().
equals
(
fieldNameLow
))
&&
if
(
// The name should be "get<FieldName>" or "<fieldName>" (for scala)
or "is<fieldName>" for boolean fields
.
(
m
.
getName
().
toLowerCase
().
equals
(
"get"
+
fieldNameLow
)
||
m
.
getName
().
toLowerCase
().
equals
(
"is"
+
fieldNameLow
)
||
m
.
getName
().
toLowerCase
().
equals
(
fieldNameLow
))
&&
// no arguments for the getter
m
.
getParameterTypes
().
length
==
0
&&
// return type is same as field type (or the generic variant of it)
(
m
.
getReturnType
().
equals
(
fieldType
)
||
(
fieldTypeGeneric
!=
null
&&
m
.
getGenericReturnType
().
equals
(
fieldTypeGeneric
))
)
(
m
.
get
Generic
ReturnType
().
equals
(
fieldType
)
||
(
fieldTypeGeneric
!=
null
&&
m
.
getGenericReturnType
().
equals
(
fieldTypeGeneric
))
)
)
{
if
(
hasGetter
)
{
throw
new
IllegalStateException
(
"Detected more than one getter"
);
...
...
@@ -1004,7 +1004,7 @@ public class TypeExtractor {
// check for setters (<FieldName>_$eq for scala)
if
((
m
.
getName
().
toLowerCase
().
equals
(
"set"
+
fieldNameLow
)
||
m
.
getName
().
toLowerCase
().
equals
(
fieldNameLow
+
"_$eq"
))
&&
m
.
getParameterTypes
().
length
==
1
&&
// one parameter of the field's type
(
m
.
getParameterTypes
()[
0
].
equals
(
fieldType
)
||
(
fieldTypeGeneric
!=
null
&&
m
.
getGenericParameterTypes
()[
0
].
equals
(
fieldTypeGeneric
)
)
)&&
(
m
.
get
Generic
ParameterTypes
()[
0
].
equals
(
fieldType
)
||
(
fieldTypeGeneric
!=
null
&&
m
.
getGenericParameterTypes
()[
0
].
equals
(
fieldTypeGeneric
)
)
)&&
// return type is void.
m
.
getReturnType
().
equals
(
Void
.
TYPE
)
)
{
...
...
flink-java/src/test/java/org/apache/flink/api/java/type/extractor/PojoTypeExtractionTest.java
浏览文件 @
63ef8e86
...
...
@@ -19,6 +19,7 @@ package org.apache.flink.api.java.type.extractor;
import
java.util.ArrayList
;
import
java.util.Arrays
;
import
java.util.Collection
;
import
java.util.Date
;
import
java.util.List
;
...
...
@@ -138,6 +139,34 @@ public class PojoTypeExtractionTest {
}
}
public
static
class
PojoWithGenericFields
{
private
Collection
<
String
>
users
;
private
boolean
favorited
;
public
boolean
isFavorited
()
{
return
favorited
;
}
public
void
setFavorited
(
boolean
favorited
)
{
this
.
favorited
=
favorited
;
}
public
Collection
<
String
>
getUsers
()
{
return
users
;
}
public
void
setUsers
(
Collection
<
String
>
users
)
{
this
.
users
=
users
;
}
}
@Test
public
void
testPojoWithGenericFields
()
{
TypeInformation
<?>
typeForClass
=
TypeExtractor
.
createTypeInfo
(
PojoWithGenericFields
.
class
);
Assert
.
assertTrue
(
typeForClass
instanceof
PojoTypeInfo
<?>);
}
// in this test, the location of the getters and setters is mixed across the type hierarchy.
public
static
class
TypedPojoGetterSetterCheck
extends
GenericPojoGetterSetterCheck
<
String
>
{
public
void
setPackageProtected
(
String
in
)
{
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录