Change in asterixdb[master]: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163

2016-09-12 Thread Yingyi Bu (Code Review) 
Yingyi Bu has posted comments on this change.

Change subject: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163
..


Patch Set 1:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/1165/1/asterixdb/asterix-app/src/test/resources/runtimets/queries/records/closed-nullable-fields_issue1616/closed-nullable-fileds_issue1616.3.query.aql
File 
asterixdb/asterix-app/src/test/resources/runtimets/queries/records/closed-nullable-fields_issue1616/closed-nullable-fileds_issue1616.3.query.aql:

Line 28: return $x
Adds an order by to make the result ordering deterministic.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7e88adbfa2721fa589e1d0564bb9e0889b722e2e
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: Yes

Change in asterixdb[master]: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163

2016-09-12 Thread Yingyi Bu (Code Review) 
Yingyi Bu has posted comments on this change.

Change subject: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163
..


Patch Set 1:

(2 comments)

https://asterix-gerrit.ics.uci.edu/#/c/1165/1//COMMIT_MSG
Commit Message:

Line 8: 
"ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163"

-->

"ASTERIXDB-1334 and ASTERIXDB-1616: Add a regression test".


https://asterix-gerrit.ics.uci.edu/#/c/1165/1/asterixdb/asterix-app/src/test/resources/runtimets/queries/records/closed-nullable-fields_issue1616/closed-nullable-fileds_issue1616.2.update.aql
File 
asterixdb/asterix-app/src/test/resources/runtimets/queries/records/closed-nullable-fields_issue1616/closed-nullable-fileds_issue1616.2.update.aql:

Line 27: load dataset Tweets using localfs 
WS.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7e88adbfa2721fa589e1d0564bb9e0889b722e2e
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: Yes

Change in asterixdb[master]: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163

2016-09-12 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7e88adbfa2721fa589e1d0564bb9e0889b722e2e
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163

2016-09-12 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7e88adbfa2721fa589e1d0564bb9e0889b722e2e
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163

2016-09-12 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2608/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7e88adbfa2721fa589e1d0564bb9e0889b722e2e
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163

2016-09-12 Thread Wail Alkowaileet (Code Review) 
Wail Alkowaileet has uploaded a new change for review.

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

Change subject: ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163
..

ASTERIXDB-1334 and ASTERIXDB-1616: Add test for the fix in 1163

Change-Id: I7e88adbfa2721fa589e1d0564bb9e0889b722e2e
---
A asterixdb/asterix-app/data/twitter/closed-nullable-fields-issue_1616.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/records/closed-nullable-fields_issue1616/closed-nullable-fileds_issue1616.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/records/closed-nullable-fields_issue1616/closed-nullable-fileds_issue1616.2.update.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/records/closed-nullable-fields_issue1616/closed-nullable-fileds_issue1616.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/records/closed-nullable-fields_issue1616/closed-nullable-fields_issue1616.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite.xml
6 files changed, 929 insertions(+), 0 deletions(-)


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

diff --git 
a/asterixdb/asterix-app/data/twitter/closed-nullable-fields-issue_1616.adm 
b/asterixdb/asterix-app/data/twitter/closed-nullable-fields-issue_1616.adm
new file mode 100644
index 000..5ca0b05
--- /dev/null
+++ b/asterixdb/asterix-app/data/twitter/closed-nullable-fields-issue_1616.adm
@@ -0,0 +1,2 @@
+{  "coordinates": {"coordinates": [1.1],"type": "string"  },  "created_at": 
"string",  "entities": {"urls": [{  "display_url": "string",  "expanded_url": 
"string",  "indices": [1],  "url": "string"}],"user_mentions": [{  "id": 1,  
"id_str": "string",  "indices": [1],  "name": "string",  "screen_name": 
"string"}]  },  "favorite_count": 1,  "favorited": true,  "filter_level": 
"string",  "geo": {"coordinates": [1.1],"type": "string"  },  "id": "000",  
"id_str": "string",  "in_reply_to_screen_name": "string",  
"in_reply_to_status_id": 1,  "in_reply_to_status_id_str": "string",  
"in_reply_to_user_id": 1,  "in_reply_to_user_id_str": "string",  
"is_quote_status": true,  "lang": "string",  "place": {"bounding_box": {  
"coordinates": [[[1.1]]],  "type": "string"},"country": 
"string","country_code": "string","full_name": "string","id": "string","name": 
"string","place_type": "string","url": "string"  },  "possibly_sensitive": 
true,  "quoted_status": {"created_at": "string","entiti
 es": {"user_mentions": [{  "id": 1,  "id_str": "string",  "indices": [1],  
"name": "string",  "screen_name": "string"}]},"favorite_count": 1,"favorited": 
true,"filter_level": "string","id": 1,"id_str": 
"string","in_reply_to_screen_name": "string","in_reply_to_status_id": 
1,"in_reply_to_status_id_str": "string","in_reply_to_user_id": 
1,"in_reply_to_user_id_str": "string","is_quote_status": true,"lang": 
"string","retweet_count": 1,"retweeted": true,"source": "string","text": 
"string","truncated": true,"user": {  "contributors_enabled": true,  
"created_at": "string",  "default_profile": true,  "default_profile_image": 
true,  "description": "string",  "favourites_count": 1,  "followers_count": 1,  
"friends_count": 1,  "geo_enabled": true,  "id": 1,  "id_str": "string",  
"is_translator": true,  "lang": "string",  "listed_count": 1,  "name": 
"string",  "profile_background_color": "string",  
"profile_background_image_url": "string",  
"profile_background_image_url_https": "string", 
  "profile_background_tile": true,  "profile_banner_url": "string",  
"profile_image_url": "string",  "profile_image_url_https": "string",  
"profile_link_color": "string",  "profile_sidebar_border_color": "string",  
"profile_sidebar_fill_color": "string",  "profile_text_color": "string",  
"profile_use_background_image": true,  "protected": true,  "screen_name": 
"string",  "statuses_count": 1,  "verified": true}  },  "quoted_status_id": 1,  
"quoted_status_id_str": "string",  "retweet_count": 1,  "retweeted": true,  
"source": "string",  "text": "string",  "timestamp_ms": "string",  "truncated": 
true,  "user": {"contributors_enabled": true,"created_at": 
"string","default_profile": true,"default_profile_image": true,"description": 
"string","favourites_count": 1,"followers_count": 1,"friends_count": 
1,"geo_enabled": true,"id": 1,"id_str": "string","is_translator": true,"lang": 
"string","listed_count": 1,"location": "string","name": 
"string","profile_background_color": "string","pro
 file_background_image_url": "string","profile_background_image_url_https": 
"string","profile_background_tile": true,"profile_banner_url": 
"string","profile_image_url": "string","profile_image_url_https": 
"string","profile_link_color": "string","profile_sidebar_border_color": 
"string","profile_sidebar_fill_color": "string","profile_text_color": 
"string","profile_use_background_image": true,"protected": true,"screen_name":