You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Till Westmann (Code Review)" <do...@asterixdb.incubator.apache.org> on 2017/07/24 18:21:32 UTC

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Till Westmann has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1897

Change subject: [ING] Add RSS test using external test feed
......................................................................

[ING] Add RSS test using external test feed

- user model changes: no
- storage format changes: no
- interface changes: no

details:
- use a simple feed from http://lorem-rss.herokuapp.com/

Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
---
M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java
1 file changed, 23 insertions(+), 0 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/97/1897/1

diff --git a/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java b/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java
index ca94a3f..5033b55 100644
--- a/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java
+++ b/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java
@@ -18,6 +18,8 @@
  */
 package org.apache.asterix.external.input.record.reader;
 
+import com.rometools.rome.feed.synd.SyndEntry;
+import org.apache.asterix.external.api.IRawRecord;
 import org.apache.asterix.external.input.record.reader.rss.RSSRecordReader;
 import org.junit.Assert;
 import org.junit.Test;
@@ -39,4 +41,25 @@
         Assert.assertNotNull(expectedException);
         Assert.assertTrue(expectedException.getMessage().contains("UnknownHostException"));
     }
+
+    private static final int NO_RECORDS = 10;
+
+    @Test
+    public void fetchFromLoremWebsite() throws MalformedURLException {
+        String dummyRssFeedURL = "http://lorem-rss.herokuapp.com/feed";
+        RSSRecordReader rssRecordReader = new RSSRecordReader(dummyRssFeedURL);
+        Exception expectedException = null;
+        int cnt = 0;
+        try {
+            while (rssRecordReader.hasNext() && cnt < NO_RECORDS) {
+                IRawRecord<SyndEntry> rec = rssRecordReader.next();
+                ++cnt;
+                Assert.assertTrue(rec.get().getTitle().startsWith("Lorem ipsum"));
+            }
+        } catch (Exception e) {
+            expectedException = e;
+        }
+        Assert.assertEquals(cnt, NO_RECORDS);
+        Assert.assertNull(expectedException);
+    }
 }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1528/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/417/ (11/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/1319/ (9/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1: BAD+1

BAD Compatibility Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1528/ : SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/1103/ (10/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/3497/ : SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/753/ (5/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/1143/ (2/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/4496/ (8/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5975/ (4/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/420/ (6/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/416/ (3/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-rebalance-cancellation/227/ (7/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Till Westmann (Code Review)" <do...@asterixdb.incubator.apache.org>.
Till Westmann has submitted this change and it was merged.

Change subject: [ING] Add RSS test using external test feed
......................................................................


[ING] Add RSS test using external test feed

- user model changes: no
- storage format changes: no
- interface changes: no

details:
- use a simple feed from http://lorem-rss.herokuapp.com/

Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1897
Sonar-Qube: Jenkins <je...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Xikui Wang <xk...@gmail.com>
---
M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java
1 file changed, 23 insertions(+), 0 deletions(-)

Approvals:
  Xikui Wang: Looks good to me, approved
  Jenkins: Verified; No violations found; Verified



diff --git a/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java b/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java
index ca94a3f..5033b55 100644
--- a/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java
+++ b/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/input/record/reader/RSSRecordReaderTest.java
@@ -18,6 +18,8 @@
  */
 package org.apache.asterix.external.input.record.reader;
 
+import com.rometools.rome.feed.synd.SyndEntry;
+import org.apache.asterix.external.api.IRawRecord;
 import org.apache.asterix.external.input.record.reader.rss.RSSRecordReader;
 import org.junit.Assert;
 import org.junit.Test;
@@ -39,4 +41,25 @@
         Assert.assertNotNull(expectedException);
         Assert.assertTrue(expectedException.getMessage().contains("UnknownHostException"));
     }
+
+    private static final int NO_RECORDS = 10;
+
+    @Test
+    public void fetchFromLoremWebsite() throws MalformedURLException {
+        String dummyRssFeedURL = "http://lorem-rss.herokuapp.com/feed";
+        RSSRecordReader rssRecordReader = new RSSRecordReader(dummyRssFeedURL);
+        Exception expectedException = null;
+        int cnt = 0;
+        try {
+            while (rssRecordReader.hasNext() && cnt < NO_RECORDS) {
+                IRawRecord<SyndEntry> rec = rssRecordReader.next();
+                ++cnt;
+                Assert.assertTrue(rec.get().getTitle().startsWith("Lorem ipsum"));
+            }
+        } catch (Exception e) {
+            expectedException = e;
+        }
+        Assert.assertEquals(cnt, NO_RECORDS);
+        Assert.assertNull(expectedException);
+    }
 }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Xikui Wang <xk...@gmail.com>

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Xikui Wang (Code Review)" <do...@asterixdb.incubator.apache.org>.
Xikui Wang has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1: Code-Review+2

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Xikui Wang <xk...@gmail.com>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/1034/ (1/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: [ING] Add RSS test using external test feed

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: [ING] Add RSS test using external test feed
......................................................................


Patch Set 1:

Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/3497/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1897
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia020052efbe0fe0de5f78017202f3f7d61b7bff3
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No