[2/2] samza git commit: SAMZA-2084: [SAMZA SQL] add logging when exceptions are swallowed and rename util package
SAMZA-2084: [SAMZA SQL] add logging when exceptions are swallowed and rename util package ## What changes were proposed in this pull request? This PR is to resolve: 1) Samza Sql Shell: add logging when exceptions are swallowed 2) Samza Sql: 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". ## How was this patch tested? Pass the current unit tests. Author: Weiqing Yang Reviewers: Srinivasulu Punuru Closes #891 from weiqingy/SAMZA-2084 Project: http://git-wip-us.apache.org/repos/asf/samza/repo Commit: http://git-wip-us.apache.org/repos/asf/samza/commit/6a1e85ee Tree: http://git-wip-us.apache.org/repos/asf/samza/tree/6a1e85ee Diff: http://git-wip-us.apache.org/repos/asf/samza/diff/6a1e85ee Branch: refs/heads/master Commit: 6a1e85eef298e3587b277bf5745d0b1d9962e4ec Parents: 4a08a23 Author: Weiqing Yang Authored: Wed Jan 23 16:51:33 2019 -0800 Committer: Srinivasulu Punuru Committed: Wed Jan 23 16:51:33 2019 -0800 -- .../samza/sql/client/impl/SamzaExecutor.java| 42 ++-- .../sql/data/SamzaSqlExecutionContext.java | 2 +- .../samza/sql/dsl/SamzaSqlDslConverter.java | 4 +- .../sql/runner/SamzaSqlApplicationConfig.java | 6 +- .../apache/samza/sql/testutil/ConfigUtil.java | 62 -- .../org/apache/samza/sql/testutil/JsonUtil.java | 91 .../samza/sql/testutil/ReflectionUtils.java | 62 -- .../samza/sql/testutil/SamzaSqlQueryParser.java | 196 - .../samza/sql/testutil/SqlFileParser.java | 103 - .../samza/sql/translator/QueryTranslator.java | 2 +- .../org/apache/samza/sql/util/ConfigUtil.java | 62 ++ .../org/apache/samza/sql/util/JsonUtil.java | 91 .../apache/samza/sql/util/ReflectionUtils.java | 62 ++ .../samza/sql/util/SamzaSqlQueryParser.java | 196 + .../apache/samza/sql/util/SqlFileParser.java| 103 + .../runner/TestSamzaSqlApplicationConfig.java | 6 +- .../runner/TestSamzaSqlApplicationRunner.java | 2 +- .../samza/sql/testutil/MyTestArrayUdf.java | 42 .../apache/samza/sql/testutil/MyTestUdf.java| 50 - .../RemoteStoreIOResolverTestFactory.java | 147 - .../sql/testutil/SampleRelConverterFactory.java | 56 - .../testutil/SampleRelTableKeyConverter.java| 39 .../SampleRelTableKeyConverterFactory.java | 41 .../samza/sql/testutil/SamzaSqlTestConfig.java | 209 --- .../sql/testutil/TestMetricsRegistryImpl.java | 117 --- .../sql/testutil/TestSamzaSqlFileParser.java| 56 - .../sql/testutil/TestSamzaSqlQueryParser.java | 75 --- .../sql/translator/TestFilterTranslator.java| 3 +- .../sql/translator/TestJoinTranslator.java | 4 +- .../sql/translator/TestProjectTranslator.java | 4 +- .../sql/translator/TestQueryTranslator.java | 8 +- .../TestSamzaSqlRemoteTableJoinFunction.java| 3 +- .../apache/samza/sql/util/MyTestArrayUdf.java | 42 .../org/apache/samza/sql/util/MyTestUdf.java| 49 + .../util/RemoteStoreIOResolverTestFactory.java | 147 + .../sql/util/SampleRelConverterFactory.java | 56 + .../sql/util/SampleRelTableKeyConverter.java| 39 .../util/SampleRelTableKeyConverterFactory.java | 40 .../samza/sql/util/SamzaSqlTestConfig.java | 209 +++ .../samza/sql/util/TestMetricsRegistryImpl.java | 117 +++ .../samza/sql/util/TestSamzaSqlFileParser.java | 56 + .../samza/sql/util/TestSamzaSqlQueryParser.java | 75 +++ .../test/samzasql/TestSamzaSqlEndToEnd.java | 8 +- .../test/samzasql/TestSamzaSqlRemoteTable.java | 6 +- .../org/apache/samza/tools/SamzaSqlConsole.java | 4 +- .../tools/benchmark/AbstractSamzaBench.java | 2 +- 46 files changed, 1398 insertions(+), 1398 deletions(-) -- http://git-wip-us.apache.org/repos/asf/samza/blob/6a1e85ee/samza-sql-shell/src/main/java/org/apache/samza/sql/client/impl/SamzaExecutor.java -- diff --git a/samza-sql-shell/src/main/java/org/apache/samza/sql/client/impl/SamzaExecutor.java b/samza-sql-shell/src/main/java/org/apache/samza/sql/client/impl/SamzaExecutor.java index 1149364..7df79c9 100755 --- a/samza-sql-shell/src/main/java/org/apache/samza/sql/client/impl/SamzaExecutor.java +++ b/samza-sql-shell/src/main/java/org/apache/samza/sql/client/impl/SamzaExecutor.java @@ -31,7 +31,6 @@ import org.apache.samza.config.*; import org.apache.samza.container.grouper.task.SingleContainerGrouperFactory; import org.apache.samza.job.ApplicationStatus; import org.apache.samza.serializers.StringSerdeFactory; -import org.apache.samza.sql.avro.AvroRelSchemaProvider; import
[jira] [Commented] (SAMZA-2084) Samza Sql: add logging when exceptions are swallowed and rename util package
[ https://issues.apache.org/jira/browse/SAMZA-2084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16750590#comment-16750590 ] ASF GitHub Bot commented on SAMZA-2084: --- Github user asfgit closed the pull request at: https://github.com/apache/samza/pull/891 > Samza Sql: add logging when exceptions are swallowed and rename util package > > > Key: SAMZA-2084 > URL: https://issues.apache.org/jira/browse/SAMZA-2084 > Project: Samza > Issue Type: Bug > Components: sql >Reporter: Weiqing Yang >Priority: Minor > Fix For: 0.15, 1.1 > > > This Jira is to track > 1) Samza Sql Shell: add logging when exceptions are swallowed > 2) Samza Sql: 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)
[jira] [Commented] (SAMZA-2084) Samza Sql: add logging when exceptions are swallowed and rename util package
[ https://issues.apache.org/jira/browse/SAMZA-2084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16750585#comment-16750585 ] ASF GitHub Bot commented on SAMZA-2084: --- GitHub user weiqingy opened a pull request: https://github.com/apache/samza/pull/891 SAMZA-2084: [Samza Sql] add logging when exceptions are swallowed and rename util package ## What changes were proposed in this pull request? This PR is to resolve: 1) Samza Sql Shell: add logging when exceptions are swallowed 2) Samza Sql: rename the package name of "testutil" to "util". ## How was this patch tested? Pass the current unit tests. You can merge this pull request into a Git repository by running: $ git pull https://github.com/weiqingy/samza SAMZA-2084 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/samza/pull/891.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 #891 commit 920a7ced5446bc580d99bb8d76402e30a129dc5d Author: Weiqing Yang Date: 2019-01-24T00:35:14Z SAMZA-2084: [Samza Sql] add logging when exceptions are swallowed and rename util package > Samza Sql: add logging when exceptions are swallowed and rename util package > > > Key: SAMZA-2084 > URL: https://issues.apache.org/jira/browse/SAMZA-2084 > Project: Samza > Issue Type: Bug > Components: sql >Reporter: Weiqing Yang >Priority: Minor > Fix For: 0.15, 1.1 > > > This Jira is to track > 1) Samza Sql Shell: add logging when exceptions are swallowed > 2) Samza Sql: 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)
[jira] [Created] (SAMZA-2084) Samza Sql: add logging when exceptions are swallowed and rename util package
Weiqing Yang created SAMZA-2084: --- Summary: Samza Sql: add logging when exceptions are swallowed and rename util package Key: SAMZA-2084 URL: https://issues.apache.org/jira/browse/SAMZA-2084 Project: Samza Issue Type: Bug Components: sql Reporter: Weiqing Yang Fix For: 0.15, 1.1 This Jira is to track 1) Samza Sql Shell: add logging when exceptions are swallowed 2) Samza Sql: 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)
[jira] [Updated] (SAMZA-2080) [VOTE] Migration of Samza git repo to gitbox.apache.org
[ https://issues.apache.org/jira/browse/SAMZA-2080?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Pawas Chhokra updated SAMZA-2080: - Description: As mandated by the Apache Infrastructure Team, all git repositories must be migrated from [git-wip-us.apache.org|http://git-wip-us.apache.org/] URL to [gitbox.apache.org|http://gitbox.apache.org/], as the old service is being decommissioned. This needs to happen before February 7th, and this ticket is to count votes for migrating Samza, on 11 AM, Jan 29, 2019. (was: As mandated by the Apache Infrastructure Team, all git repositories must be migrated from [git-wip-us.apache.org|http://git-wip-us.apache.org/] URL to [gitbox.apache.org|http://gitbox.apache.org/], as the old service is being decommissioned. This needs to happen before February 7th, and this ticket is to count votes for migrating Samza, on 11 AM, Jan 25, 2019.) > [VOTE] Migration of Samza git repo to gitbox.apache.org > --- > > Key: SAMZA-2080 > URL: https://issues.apache.org/jira/browse/SAMZA-2080 > Project: Samza > Issue Type: Task >Reporter: Pawas Chhokra >Priority: Major > > As mandated by the Apache Infrastructure Team, all git repositories must be > migrated from [git-wip-us.apache.org|http://git-wip-us.apache.org/] URL to > [gitbox.apache.org|http://gitbox.apache.org/], as the old service is being > decommissioned. This needs to happen before February 7th, and this ticket is > to count votes for migrating Samza, on 11 AM, Jan 29, 2019. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
