[
https://issues.apache.org/jira/browse/KYLIN-3630?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
jiatao.tao updated KYLIN-3630:
------------------------------
Description:
In code RawMeasureType#RawMeasureType, we can see that both funcName and
dataType are use, and this can be remove cuz there's no constraint on its
constructor.
{code:java}
@SuppressWarnings("unused")
private final DataType dataType;
public RawMeasureType(String funcName, DataType dataType) {
this.dataType = dataType;
}
{code}
And after we done that, we can mo easily test MeasureType without passing in
meaningless args.
Like codes in org.apache.kylin.measure.dim.DimCountDistinctMeasureType, other
{code:java}
@Override
public MeasureType<Object> createMeasureType(String funcName, DataType
dataType) {
return new DimCountDistinctMeasureType();
}
{code}
was:
Like in code org.apache.kylin.measure.raw.RawMeasureType#RawMeasureType, we can
see that both funcName and dataType are use, and this can be remove cuz
there's no constraint on its constructor.
And after we done that, we can mo easily test MeasureType without passing in
meaningless args.
Like codes in org.apache.kylin.measure.dim.DimCountDistinctMeasureType
> remove unused fields in the implementations of MeasureType
> ----------------------------------------------------------
>
> Key: KYLIN-3630
> URL: https://issues.apache.org/jira/browse/KYLIN-3630
> Project: Kylin
> Issue Type: Improvement
> Reporter: jiatao.tao
> Assignee: jiatao.tao
> Priority: Minor
>
> In code RawMeasureType#RawMeasureType, we can see that both funcName and
> dataType are use, and this can be remove cuz there's no constraint on its
> constructor.
> {code:java}
> @SuppressWarnings("unused")
> private final DataType dataType;
> public RawMeasureType(String funcName, DataType dataType) {
> this.dataType = dataType;
> }
> {code}
> And after we done that, we can mo easily test MeasureType without passing in
> meaningless args.
> Like codes in org.apache.kylin.measure.dim.DimCountDistinctMeasureType, other
> {code:java}
> @Override
> public MeasureType<Object> createMeasureType(String funcName,
> DataType dataType) {
> return new DimCountDistinctMeasureType();
> }
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
