You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Murtadha Hubail (Code Review)" <do...@asterixdb.incubator.apache.org> on 2016/02/06 03:53:42 UTC
Change in asterixdb[master]: Remove unnecessary deprecation SuppressWarnings
Murtadha Hubail has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/618
Change subject: Remove unnecessary deprecation SuppressWarnings
......................................................................
Remove unnecessary deprecation SuppressWarnings
Change-Id: I94eed0d97d9c3137b128a79b7e6e5e8ca00913a3
---
M asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java
M asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
M asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java
M asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java
4 files changed, 5 insertions(+), 8 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/18/618/1
diff --git a/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java b/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java
index e950da5..1625c2b 100644
--- a/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java
+++ b/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java
@@ -36,8 +36,6 @@
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
-import junit.extensions.PA;
-
import org.apache.asterix.feed.CentralFeedManager;
import org.apache.asterix.metadata.MetadataManager;
import org.apache.asterix.metadata.MetadataTransactionContext;
@@ -58,7 +56,8 @@
import org.junit.Assert;
import org.junit.Test;
-@SuppressWarnings("deprecation")
+import junit.extensions.PA;
+
public class ConnectorAPIServletTest {
@Test
diff --git a/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java b/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
index cf9665b..120fdfd 100644
--- a/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
+++ b/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
@@ -44,7 +44,6 @@
import org.junit.Assert;
import org.junit.Test;
-@SuppressWarnings("deprecation")
public class VersionAPIServletTest {
@Test
diff --git a/asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java b/asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java
index a397875..532cea9 100644
--- a/asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java
+++ b/asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java
@@ -28,8 +28,6 @@
import java.util.Arrays;
import java.util.List;
-import junit.extensions.PA;
-
import org.apache.asterix.api.common.SessionConfig;
import org.apache.asterix.common.config.AsterixExternalProperties;
import org.apache.asterix.compiler.provider.AqlCompilationProvider;
@@ -42,7 +40,9 @@
import org.junit.Assert;
import org.junit.Test;
-@SuppressWarnings({ "unchecked", "deprecation" })
+import junit.extensions.PA;
+
+@SuppressWarnings({ "unchecked" })
public class QueryTranslatorTest {
@Test
diff --git a/asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java b/asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java
index 2993b87..a0a5c5f 100644
--- a/asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java
+++ b/asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java
@@ -31,7 +31,6 @@
import org.junit.Assert;
import org.junit.Test;
-@SuppressWarnings("deprecation")
public class JSONDeserializerForTypesTest {
@Test
--
To view, visit https://asterix-gerrit.ics.uci.edu/618
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I94eed0d97d9c3137b128a79b7e6e5e8ca00913a3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <hu...@gmail.com>
Change in asterixdb[master]: Remove unnecessary deprecation SuppressWarnings
Posted by "Till Westmann (Code Review)" <do...@asterixdb.incubator.apache.org>.
Till Westmann has posted comments on this change.
Change subject: Remove unnecessary deprecation SuppressWarnings
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/618
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I94eed0d97d9c3137b128a79b7e6e5e8ca00913a3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <hu...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No
Change in asterixdb[master]: Remove unnecessary deprecation SuppressWarnings
Posted by "Murtadha Hubail (Code Review)" <do...@asterixdb.incubator.apache.org>.
Murtadha Hubail has submitted this change and it was merged.
Change subject: Remove unnecessary deprecation SuppressWarnings
......................................................................
Remove unnecessary deprecation SuppressWarnings
Change-Id: I94eed0d97d9c3137b128a79b7e6e5e8ca00913a3
Reviewed-on: https://asterix-gerrit.ics.uci.edu/618
Reviewed-by: Till Westmann <ti...@apache.org>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
---
M asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java
M asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
M asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java
M asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java
4 files changed, 5 insertions(+), 8 deletions(-)
Approvals:
Till Westmann: Looks good to me, approved
Jenkins: Verified
diff --git a/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java b/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java
index e950da5..1625c2b 100644
--- a/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java
+++ b/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java
@@ -36,8 +36,6 @@
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
-import junit.extensions.PA;
-
import org.apache.asterix.feed.CentralFeedManager;
import org.apache.asterix.metadata.MetadataManager;
import org.apache.asterix.metadata.MetadataTransactionContext;
@@ -58,7 +56,8 @@
import org.junit.Assert;
import org.junit.Test;
-@SuppressWarnings("deprecation")
+import junit.extensions.PA;
+
public class ConnectorAPIServletTest {
@Test
diff --git a/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java b/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
index cf9665b..120fdfd 100644
--- a/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
+++ b/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
@@ -44,7 +44,6 @@
import org.junit.Assert;
import org.junit.Test;
-@SuppressWarnings("deprecation")
public class VersionAPIServletTest {
@Test
diff --git a/asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java b/asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java
index a397875..532cea9 100644
--- a/asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java
+++ b/asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java
@@ -28,8 +28,6 @@
import java.util.Arrays;
import java.util.List;
-import junit.extensions.PA;
-
import org.apache.asterix.api.common.SessionConfig;
import org.apache.asterix.common.config.AsterixExternalProperties;
import org.apache.asterix.compiler.provider.AqlCompilationProvider;
@@ -42,7 +40,9 @@
import org.junit.Assert;
import org.junit.Test;
-@SuppressWarnings({ "unchecked", "deprecation" })
+import junit.extensions.PA;
+
+@SuppressWarnings({ "unchecked" })
public class QueryTranslatorTest {
@Test
diff --git a/asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java b/asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java
index 2993b87..a0a5c5f 100644
--- a/asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java
+++ b/asterix-om/src/test/java/org/apache/asterix/om/util/JSONDeserializerForTypesTest.java
@@ -31,7 +31,6 @@
import org.junit.Assert;
import org.junit.Test;
-@SuppressWarnings("deprecation")
public class JSONDeserializerForTypesTest {
@Test
--
To view, visit https://asterix-gerrit.ics.uci.edu/618
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I94eed0d97d9c3137b128a79b7e6e5e8ca00913a3
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <hu...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Murtadha Hubail <hu...@gmail.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Change in asterixdb[master]: Remove unnecessary deprecation SuppressWarnings
Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.
Change subject: Remove unnecessary deprecation SuppressWarnings
......................................................................
Patch Set 1:
Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/800/
--
To view, visit https://asterix-gerrit.ics.uci.edu/618
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I94eed0d97d9c3137b128a79b7e6e5e8ca00913a3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <hu...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No
Change in asterixdb[master]: Remove unnecessary deprecation SuppressWarnings
Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.
Change subject: Remove unnecessary deprecation SuppressWarnings
......................................................................
Patch Set 1: Verified+1
Build Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/800/ : SUCCESS
--
To view, visit https://asterix-gerrit.ics.uci.edu/618
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I94eed0d97d9c3137b128a79b7e6e5e8ca00913a3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <hu...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No