[
https://issues.apache.org/jira/browse/SAMZA-1960?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Weiqing Yang updated SAMZA-1960:
--------------------------------
Description:
This Jira is to track the follow issues:
# Currently, the unit tests about "QueryTranslator" are using their own
version of
[translate|https://github.com/apache/samza/blob/master/samza-sql/src/main/java/org/apache/samza/sql/translator/QueryTranslator.java#L107](),
the logic of which is a little messy. We should either use the translate()
[here|https://github.com/apache/samza/blob/master/samza-sql/src/main/java/org/apache/samza/sql/translator/QueryTranslator.java#L126]
or refactor the test version of
[translate|https://github.com/apache/samza/blob/master/samza-sql/src/main/java/org/apache/samza/sql/translator/QueryTranslator.java#L107]()
and its related unit tests.
# We should rename the package name of
"[testutil|https://github.com/apache/samza/tree/master/samza-sql/src/main/java/org/apache/samza/sql/testutil]";
to "util".
was:
This Jira is to tracking the follow issues:
# Currently, the unit tests about "QueryTranslator" are using their own
version of
[translate|https://github.com/apache/samza/blob/master/samza-sql/src/main/java/org/apache/samza/sql/translator/QueryTranslator.java#L107](),
the logic of which is a little messy. We should either use the translate()
[here|https://github.com/apache/samza/blob/master/samza-sql/src/main/java/org/apache/samza/sql/translator/QueryTranslator.java#L126]
or refactor the test version of
[translate|https://github.com/apache/samza/blob/master/samza-sql/src/main/java/org/apache/samza/sql/translator/QueryTranslator.java#L107]()
and its related unit tests.
# We should rename the package name of
"[testutil|https://github.com/apache/samza/tree/master/samza-sql/src/main/java/org/apache/samza/sql/testutil]";
to "util".
> Refactor QueryTranslator unit tests
> -----------------------------------
>
> Key: SAMZA-1960
> URL: https://issues.apache.org/jira/browse/SAMZA-1960
> Project: Samza
> Issue Type: Bug
> Components: sql
> Reporter: Weiqing Yang
> Priority: Major
> Fix For: 0.15.0
>
>
> This Jira is to track the follow issues:
> # Currently, the unit tests about "QueryTranslator" are using their own
> version of
> [translate|https://github.com/apache/samza/blob/master/samza-sql/src/main/java/org/apache/samza/sql/translator/QueryTranslator.java#L107](),
> the logic of which is a little messy. We should either use the translate()
> [here|https://github.com/apache/samza/blob/master/samza-sql/src/main/java/org/apache/samza/sql/translator/QueryTranslator.java#L126]
> or refactor the test version of
> [translate|https://github.com/apache/samza/blob/master/samza-sql/src/main/java/org/apache/samza/sql/translator/QueryTranslator.java#L107]()
> and its related unit tests.
> # We should rename the package name of
> "[testutil|https://github.com/apache/samza/tree/master/samza-sql/src/main/java/org/apache/samza/sql/testutil]";
> to "util".
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
