[jira] [Assigned] (ASTERIXDB-2381) System instability on Mac is back - occasional unusable cluster state

2018-05-01 Thread Michael J. Carey (JIRA)

 [ 
https://issues.apache.org/jira/browse/ASTERIXDB-2381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael J. Carey reassigned ASTERIXDB-2381:
---

Assignee: Murtadha Hubail

> System instability on Mac is back - occasional unusable cluster state
> -
>
> Key: ASTERIXDB-2381
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-2381
> Project: Apache AsterixDB
>  Issue Type: Bug
>  Components: *DB - AsterixDB, ADM - Admin, NET - Network
>Affects Versions: 0.9.4
>Reporter: Michael J. Carey
>Assignee: Murtadha Hubail
>Priority: Major
> Fix For: 0.9.4
>
> Attachments: logs.zip
>
>
> The old Mac problem of an unusable cluster state seems to be back.  Maybe due 
> to sleeps and network changes, maybe due to a too-short timeout, not sure - 
> but it's happening too frequently in the U-Washington snapshot.  Logs 
> attached from my Mac - happened to me yesterday.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (ASTERIXDB-2381) System instability on Mac is back - occasional unusable cluster state

2018-05-01 Thread Michael J. Carey (JIRA)
Michael J. Carey created ASTERIXDB-2381:
---

 Summary: System instability on Mac is back - occasional unusable 
cluster state
 Key: ASTERIXDB-2381
 URL: https://issues.apache.org/jira/browse/ASTERIXDB-2381
 Project: Apache AsterixDB
  Issue Type: Bug
  Components: *DB - AsterixDB, ADM - Admin, NET - Network
Affects Versions: 0.9.4
Reporter: Michael J. Carey
 Fix For: 0.9.4
 Attachments: logs.zip

The old Mac problem of an unusable cluster state seems to be back.  Maybe due 
to sleeps and network changes, maybe due to a too-short timeout, not sure - but 
it's happening too frequently in the U-Washington snapshot.  Logs attached from 
my Mac - happened to me yesterday.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


Change in asterixdb[master]: [ASTERIXDB-2344] LIMIT pushdown for primary index

2018-05-01 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change.

Change subject: [ASTERIXDB-2344] LIMIT pushdown for primary index
..


Patch Set 17: Code-Review+1

LGTM. I only have a few minor comments.

#1. Do we want to keep the optimizer tests? or just use runtime test framework? 
It looks like you are using the runtime test to generate the plan and check it 
without introducing the optimizer tests.

#2. Can you add a description to each test? What are they supposed to check?

#3. Before declaring outputLimit, selectCondition, and tupleFilterFactory 
variables in each class, can we add a simple comment about them? What's the 
purpose of them?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I824fcad79995325e12a1a81d629160025294b915
Gerrit-PatchSet: 17
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Dmitry Lychagin 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Luo Chen 
Gerrit-Reviewer: Taewoo Kim (please use wangs...@gmail.com) 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2: Contrib+1

Analytics Compatibility Tests Successful
https://goo.gl/sxkYM7 : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2: Contrib+1

BAD Compatibility Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Analytics Compatibility Compilation Successful
https://goo.gl/oNqCmx : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Dmitry Lychagin (Code Review)
Hello Anon. E. Moose #1000171, Till Westmann, Jenkins,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#2).

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..

[ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

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

Details:
- Make "/" operator return double if both operands are integers
- Add "DIV" operator which is the same as "/" except that
  it returns integer if both operands are integers
  (as "/" did before this change)
- Add "MOD" operator which is an alias for "%" operator
- Remove "IDIV" operator from the grammar

Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/numeric/divide_int16/divide_int16.2.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/numeric/divide_int32/divide_int32.2.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/numeric/divide_int64/divide_int64.2.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/numeric/divide_int8/divide_int8.2.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/divide_int16/divide_int16.2.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/divide_int32/divide_int32.2.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/divide_int64/divide_int64.2.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/divide_int8/divide_int8.2.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/custord/customer_q_04/customer_q_04.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/custord/customer_q_05/customer_q_05.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int16/divide_int16.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int16/divide_int16.2.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int32/divide_int32.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int32/divide_int32.2.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int64/divide_int64.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int64/divide_int64.2.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int8/divide_int8.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int8/divide_int8.2.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/numeric/divide_int16/divide_int16.2.ast
A 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/numeric/divide_int32/divide_int32.2.ast
A 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/numeric/divide_int64/divide_int64.2.ast
A 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/numeric/divide_int8/divide_int8.2.ast
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/string/substr01/substr01.3.ast
M asterixdb/asterix-doc/src/main/markdown/sqlpp/2_expr.md
M asterixdb/asterix-doc/src/site/markdown/aql/manual.md
M asterixdb/asterix-lang-aql/src/main/javacc/AQL.jj
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/expression/OperatorExpr.java
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/struct/OperatorType.java
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/util/CommonFunctionMapUtil.java
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/BuiltinFunctions.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/NumericDivideTypeComputer.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/AbstractNumericArithmeticEval.java
C 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericDivDescriptor.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericDivideDescriptor.java
R 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericPowerDescriptor.java
M 
ast

Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/8939/ (1/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/3388/ 
(8/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/4008/ (10/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/3928/ 
(6/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/3435/ 
(12/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/1483/ 
(2/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/3907/
 (4/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/3476/
 (3/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/7415/ (7/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


[jira] [Resolved] (ASTERIXDB-2376) Support negative offset for substring()

2018-05-01 Thread Dmitry Lychagin (JIRA)

 [ 
https://issues.apache.org/jira/browse/ASTERIXDB-2376?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dmitry Lychagin resolved ASTERIXDB-2376.

Resolution: Fixed

> Support negative offset for substring()
> ---
>
> Key: ASTERIXDB-2376
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-2376
> Project: Apache AsterixDB
>  Issue Type: Improvement
>  Components: FUN - Functions
>Reporter: Dmitry Lychagin
>Assignee: Dmitry Lychagin
>Priority: Minor
>
> When negative offset is passed to substring() function it should mean: count 
> from the end of the string 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (ASTERIXDB-2376) Support negative offset for substring()

2018-05-01 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/ASTERIXDB-2376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16460364#comment-16460364
 ] 

ASF subversion and git services commented on ASTERIXDB-2376:


Commit d70b41ceb862917eaf0a5b1dd9ae036413e2460c in asterixdb's branch 
refs/heads/master from [~dlychagin-cb]
[ https://git-wip-us.apache.org/repos/asf?p=asterixdb.git;h=d70b41c ]

[ASTERIXDB-2376][FUN] Negative offset for substring()

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

Details:
- Support negative offset as an argument to substring()
  (that means: count from the end of the string)

Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2626
Sonar-Qube: Jenkins 
Tested-by: Jenkins 
Contrib: Jenkins 
Integration-Tests: Jenkins 
Reviewed-by: Till Westmann 


> Support negative offset for substring()
> ---
>
> Key: ASTERIXDB-2376
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-2376
> Project: Apache AsterixDB
>  Issue Type: Improvement
>  Components: FUN - Functions
>Reporter: Dmitry Lychagin
>Assignee: Dmitry Lychagin
>Priority: Minor
>
> When negative offset is passed to substring() function it should mean: count 
> from the end of the string 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()

2018-05-01 Thread Dmitry Lychagin (Code Review)
Dmitry Lychagin has submitted this change and it was merged.

Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring()
..


[ASTERIXDB-2376][FUN] Negative offset for substring()

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

Details:
- Support negative offset as an argument to substring()
  (that means: count from the end of the string)

Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2626
Sonar-Qube: Jenkins 
Tested-by: Jenkins 
Contrib: Jenkins 
Integration-Tests: Jenkins 
Reviewed-by: Till Westmann 
---
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substring2-4/substring2-4.3.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/string/substr01/substr01.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/string/substring2-4/substring2-4.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/stringoffset/substring/substring.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/stringoffset/substring2/substring2.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/string/substr01/substr01.3.ast
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/string/substring2-4/substring2-4.3.ast
M asterixdb/asterix-doc/src/main/markdown/builtins/2_string_common.md
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/Substring2Descriptor.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/SubstringDescriptor.java
13 files changed, 18 insertions(+), 17 deletions(-)

Approvals:
  Anon. E. Moose #1000171: 
  Till Westmann: Looks good to me, approved
  Jenkins: Verified; No violations found; ; Verified



diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql
 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql
index bc9e7f7..a45b24c 100644
--- 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql
+++ 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql
@@ -36,7 +36,7 @@
 let $str8:=substring($str7,0)
 
 let $str9:="This is a test string"
-let $str10:=substring($str9,-1)
+let $str10:=substring($str9,-6)
 
 let $str11:="This is a test string"
 let $str12:="This is a another test string"
diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql
 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql
index 075af82..ecfcd94 100644
--- 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql
+++ 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql
@@ -18,5 +18,5 @@
  */
 use dataverse test;
 
-let $c1 := substring("HEllow",-1)
+let $c1 := substring("HEllow",-3)
 return {"result1": $c1}
diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp
 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp
index b9c79d0..6abe9ff 100644
--- 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp
+++ 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp
@@ -23,4 +23,4 @@
  * Date   : 18th April 2012
  */
 
-{'str2':substring('Hello World',9),'str4':substring('This is a test 
string',20),'str6':substring('This is a test string',21),'str8':substring('This 
is a test string',0),'str10':substring('This is a test 
string',-1),'str13':substring(`string-concat`(['This is a test string','This is 
a another test string']),20),'str14':substring('UC Irvine',(`string-length`('UC 
Irvine') / 2 - 1))};
+{'str2':substring('Hello World',9),'str4':substring('This is a test 
string',20),'str6':substring('This is a test string',21),'str8':substring('This 
is a test string',0),'str10':substring('This is a test 
string',-6),'str13':substring(`string-concat`(['This is a test string','This is 
a another test string']),20),'str14':substring('UC Irvine',(`string-length`('UC 
Irvine') / 2 - 1))};
diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substring2-4/substring2-4.3.query.sqlpp
 
b/asterixdb/asterix-app/src/

Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1: Code-Review+2

(2 comments)

https://asterix-gerrit.ics.uci.edu/#/c/2630/1/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java:

PS1, Line 92: that retain the original field variable
What does this mean?


https://asterix-gerrit.ics.uci.edu/#/c/2630/1/asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int8/divide_int8.1.adm
File 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int8/divide_int8.1.adm:

PS1, Line 1: 1.0
Should we change the printers to not print ".0" here?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: Yes


Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()

2018-05-01 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change.

Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring()
..


Patch Set 2: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


[jira] [Updated] (ASTERIXDB-2380) Join with additional predicate in subquery returns different results.

2018-05-01 Thread Xikui Wang (JIRA)

 [ 
https://issues.apache.org/jira/browse/ASTERIXDB-2380?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Xikui Wang updated ASTERIXDB-2380:
--
Summary: Join with additional predicate in subquery returns different 
results.  (was: Join with additional predicates in subquery returns different 
results.)

> Join with additional predicate in subquery returns different results.
> -
>
> Key: ASTERIXDB-2380
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-2380
> Project: Apache AsterixDB
>  Issue Type: Bug
>Reporter: Xikui Wang
>Priority: Major
> Attachments: country.adm, mountain.adm, sea.adm, starter.sqlpp
>
>
> There is query issue that popped up when we were helping UW students using 
> AsterixDB. The following query looks for countries that share mountain and 
> sea. One problem about this query is if add another predicate which compares 
> a field having the same value, the result changes. The query is as follow:
> {code}
> USE hw5index;
> WITH commons AS (SELECT m1c AS first_country, m2c AS second_country,
> m1.name AS mountain, s1.name AS sea
>  FROM mountain AS m1, sea AS s1, mountain AS m2, sea AS s2,
>   split(m1.`-country`, " ") AS m1c, split(s1.`-country`, 
> " ") AS s1c,
>   split(m2.`-country`, " ") AS m2c, split(s2.`-country`, 
> " ") AS s2c
>  WHERE m1c = s1c AND m2c = s2c AND m1c != m2c AND m1.name = 
> m2.name AND
>s1.name = s2.name AND m1c > m2c)
> SELECT DISTINCT c.first_country, c.second_country,
>(SELECT VALUE c2
> FROM commons AS c2
>where c2.first_country = c.first_country) AS mountains
> FROM commons AS c;
> {code}
> Query with an additional predicate
> {code}
> USE hw5index;
> WITH commons AS (SELECT m1c AS first_country, m2c AS second_country,
> m1.name AS mountain, s1.name AS sea
>  FROM mountain AS m1, sea AS s1, mountain AS m2, sea AS s2,
>   split(m1.`-country`, " ") AS m1c, split(s1.`-country`, 
> " ") AS s1c,
>   split(m2.`-country`, " ") AS m2c, split(s2.`-country`, 
> " ") AS s2c
>  WHERE m1c = s1c AND m2c = s2c AND m1c != m2c AND m1.name = 
> m2.name AND
>s1.name = s2.name AND m1c > m2c)
> SELECT DISTINCT c.first_country, c.second_country,
>(SELECT VALUE c2
> FROM commons AS c2
>where c2.first_country = c.first_country AND 
> c2.second_country = c.second_country) AS mountains
> FROM commons AS c;
> {code}
> The DDL and data are attached



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (ASTERIXDB-2380) Join with additional predicate in subquery returns different results.

2018-05-01 Thread Xikui Wang (JIRA)

 [ 
https://issues.apache.org/jira/browse/ASTERIXDB-2380?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Xikui Wang updated ASTERIXDB-2380:
--
Attachment: starter.sqlpp
sea.adm
mountain.adm
country.adm

> Join with additional predicate in subquery returns different results.
> -
>
> Key: ASTERIXDB-2380
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-2380
> Project: Apache AsterixDB
>  Issue Type: Bug
>Reporter: Xikui Wang
>Priority: Major
> Attachments: country.adm, mountain.adm, sea.adm, starter.sqlpp
>
>
> There is query issue that popped up when we were helping UW students using 
> AsterixDB. The following query looks for countries that share mountain and 
> sea. One problem about this query is if add another predicate which compares 
> a field having the same value, the result changes. The query is as follow:
> {code}
> USE hw5index;
> WITH commons AS (SELECT m1c AS first_country, m2c AS second_country,
> m1.name AS mountain, s1.name AS sea
>  FROM mountain AS m1, sea AS s1, mountain AS m2, sea AS s2,
>   split(m1.`-country`, " ") AS m1c, split(s1.`-country`, 
> " ") AS s1c,
>   split(m2.`-country`, " ") AS m2c, split(s2.`-country`, 
> " ") AS s2c
>  WHERE m1c = s1c AND m2c = s2c AND m1c != m2c AND m1.name = 
> m2.name AND
>s1.name = s2.name AND m1c > m2c)
> SELECT DISTINCT c.first_country, c.second_country,
>(SELECT VALUE c2
> FROM commons AS c2
>where c2.first_country = c.first_country) AS mountains
> FROM commons AS c;
> {code}
> Query with an additional predicate
> {code}
> USE hw5index;
> WITH commons AS (SELECT m1c AS first_country, m2c AS second_country,
> m1.name AS mountain, s1.name AS sea
>  FROM mountain AS m1, sea AS s1, mountain AS m2, sea AS s2,
>   split(m1.`-country`, " ") AS m1c, split(s1.`-country`, 
> " ") AS s1c,
>   split(m2.`-country`, " ") AS m2c, split(s2.`-country`, 
> " ") AS s2c
>  WHERE m1c = s1c AND m2c = s2c AND m1c != m2c AND m1.name = 
> m2.name AND
>s1.name = s2.name AND m1c > m2c)
> SELECT DISTINCT c.first_country, c.second_country,
>(SELECT VALUE c2
> FROM commons AS c2
>where c2.first_country = c.first_country AND 
> c2.second_country = c.second_country) AS mountains
> FROM commons AS c;
> {code}
> The DDL and data are attached



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (ASTERIXDB-2380) Join with additional predicates in subquery returns different results.

2018-05-01 Thread Xikui Wang (JIRA)
Xikui Wang created ASTERIXDB-2380:
-

 Summary: Join with additional predicates in subquery returns 
different results.
 Key: ASTERIXDB-2380
 URL: https://issues.apache.org/jira/browse/ASTERIXDB-2380
 Project: Apache AsterixDB
  Issue Type: Bug
Reporter: Xikui Wang


There is query issue that popped up when we were helping UW students using 
AsterixDB. The following query looks for countries that share mountain and sea. 
One problem about this query is if add another predicate which compares a field 
having the same value, the result changes. The query is as follow:

{code}
USE hw5index;

WITH commons AS (SELECT m1c AS first_country, m2c AS second_country,
m1.name AS mountain, s1.name AS sea
 FROM mountain AS m1, sea AS s1, mountain AS m2, sea AS s2,
  split(m1.`-country`, " ") AS m1c, split(s1.`-country`, " 
") AS s1c,
  split(m2.`-country`, " ") AS m2c, split(s2.`-country`, " 
") AS s2c
 WHERE m1c = s1c AND m2c = s2c AND m1c != m2c AND m1.name = 
m2.name AND
   s1.name = s2.name AND m1c > m2c)

SELECT DISTINCT c.first_country, c.second_country,
   (SELECT VALUE c2
FROM commons AS c2
   where c2.first_country = c.first_country) AS mountains
FROM commons AS c;
{code}

Query with an additional predicate
{code}
USE hw5index;

WITH commons AS (SELECT m1c AS first_country, m2c AS second_country,
m1.name AS mountain, s1.name AS sea
 FROM mountain AS m1, sea AS s1, mountain AS m2, sea AS s2,
  split(m1.`-country`, " ") AS m1c, split(s1.`-country`, " 
") AS s1c,
  split(m2.`-country`, " ") AS m2c, split(s2.`-country`, " 
") AS s2c
 WHERE m1c = s1c AND m2c = s2c AND m1c != m2c AND m1.name = 
m2.name AND
   s1.name = s2.name AND m1c > m2c)

SELECT DISTINCT c.first_country, c.second_country,
   (SELECT VALUE c2
FROM commons AS c2
   where c2.first_country = c.first_country AND 
c2.second_country = c.second_country) AS mountains
FROM commons AS c;
{code}

The DDL and data are attached



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Murtadha Hubail (Code Review)
Murtadha Hubail has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1: Contrib+1

BAD Compatibility Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1: Contrib+1

Analytics Compatibility Tests Successful
https://goo.gl/pU5BEo : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5: Contrib+1

Analytics Compatibility Tests Successful
https://goo.gl/wUDuW8 : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

BAD Compatibility Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


[jira] [Closed] (ASTERIXDB-2379) Inconsistent naming for LSM Component timestamps

2018-05-01 Thread Chen Luo (JIRA)

 [ 
https://issues.apache.org/jira/browse/ASTERIXDB-2379?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chen Luo closed ASTERIXDB-2379.
---
Resolution: Not A Problem

> Inconsistent naming for LSM Component timestamps
> 
>
> Key: ASTERIXDB-2379
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-2379
> Project: Apache AsterixDB
>  Issue Type: Improvement
>  Components: STO - Storage
>Reporter: Chen Luo
>Priority: Major
>
> The current naming of LSM components are inconsistent. 
> AbstractLSMIndexFileManager.getComponentStartTime/getComponentEndTime assumes 
> a component is named as "beginTS-endTS". However, physically a component is 
> named as "endTS-beginTS". This leads to recovery problems because certain 
> valid components could be erroneously deleted because of index checkpointing 
> check.
> The problem is caused by merge operation. For example, in 
> LSMBTree.getMergeFileReferences, firstComponent is actually newer than 
> lastComponent, which breaks the assumption of time range.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (ASTERIXDB-2379) Inconsistent naming for LSM Component timestamps

2018-05-01 Thread Chen Luo (JIRA)

 [ 
https://issues.apache.org/jira/browse/ASTERIXDB-2379?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chen Luo updated ASTERIXDB-2379:

Description: 
The current naming of LSM components are inconsistent. 
AbstractLSMIndexFileManager.getComponentStartTime/getComponentEndTime assumes a 
component is named as "beginTS-endTS". However, physically a component is named 
as "endTS-beginTS". This leads to recovery problems because certain valid 
components could be erroneously deleted because of index checkpointing check.

The problem is caused by merge operation. For example, in 
LSMBTree.getMergeFileReferences, firstComponent is actually newer than 
lastComponent, which breaks the assumption of time range.

  was:
The current naming of LSM components are inconsistent. 
AbstractLSMIndexFileManager.getComponentStartTime/getComponentEndTime assumes a 
component is named as "beginTS-endTS". However, physically a component is named 
as "endTS-beginTS".

The problem is caused by merge operation. For example, in 
LSMBTree.getMergeFileReferences, firstComponent is actually newer than 
lastComponent, which breaks the assumption of time range.


> Inconsistent naming for LSM Component timestamps
> 
>
> Key: ASTERIXDB-2379
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-2379
> Project: Apache AsterixDB
>  Issue Type: Improvement
>  Components: STO - Storage
>Reporter: Chen Luo
>Priority: Major
>
> The current naming of LSM components are inconsistent. 
> AbstractLSMIndexFileManager.getComponentStartTime/getComponentEndTime assumes 
> a component is named as "beginTS-endTS". However, physically a component is 
> named as "endTS-beginTS". This leads to recovery problems because certain 
> valid components could be erroneously deleted because of index checkpointing 
> check.
> The problem is caused by merge operation. For example, in 
> LSMBTree.getMergeFileReferences, firstComponent is actually newer than 
> lastComponent, which breaks the assumption of time range.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (ASTERIXDB-2379) Inconsistent naming for LSM Component timestamps

2018-05-01 Thread Chen Luo (JIRA)
Chen Luo created ASTERIXDB-2379:
---

 Summary: Inconsistent naming for LSM Component timestamps
 Key: ASTERIXDB-2379
 URL: https://issues.apache.org/jira/browse/ASTERIXDB-2379
 Project: Apache AsterixDB
  Issue Type: Improvement
  Components: STO - Storage
Reporter: Chen Luo


The current naming of LSM components are inconsistent. 
AbstractLSMIndexFileManager.getComponentStartTime/getComponentEndTime assumes a 
component is named as "beginTS-endTS". However, physically a component is named 
as "endTS-beginTS".

The problem is caused by merge operation. For example, in 
LSMBTree.getMergeFileReferences, firstComponent is actually newer than 
lastComponent, which breaks the assumption of time range.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Analytics Compatibility Compilation Successful
https://goo.gl/2eZ5fn : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/3463/ 
(9/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/3434/ 
(11/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/3387/ 
(6/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/3906/
 (3/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/8938/ (1/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/1482/ 
(4/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/7414/ (10/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/4007/ (8/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Dmitry Lychagin (Code Review)
Dmitry Lychagin has uploaded a new change for review.

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

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..

[ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

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

Details:
- Make "/" operator return double if both operands are integers
- Add "DIV" operator which is the same as "/" except that
  it returns integer if both operands are integers
  (as "/" did before this change)
- Add "MOD" operator which is an alias for "%" operator
- Remove "IDIV" operator from the grammar

Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/numeric/divide_int16/divide_int16.2.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/numeric/divide_int32/divide_int32.2.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/numeric/divide_int64/divide_int64.2.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/numeric/divide_int8/divide_int8.2.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/divide_int16/divide_int16.2.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/divide_int32/divide_int32.2.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/divide_int64/divide_int64.2.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/divide_int8/divide_int8.2.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/custord/customer_q_04/customer_q_04.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/custord/customer_q_05/customer_q_05.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int16/divide_int16.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int16/divide_int16.2.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int32/divide_int32.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int32/divide_int32.2.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int64/divide_int64.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int64/divide_int64.2.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int8/divide_int8.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/divide_int8/divide_int8.2.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/numeric/divide_int16/divide_int16.2.ast
A 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/numeric/divide_int32/divide_int32.2.ast
A 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/numeric/divide_int64/divide_int64.2.ast
A 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/numeric/divide_int8/divide_int8.2.ast
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/string/substr01/substr01.3.ast
M asterixdb/asterix-doc/src/main/markdown/sqlpp/2_expr.md
M asterixdb/asterix-doc/src/site/markdown/aql/manual.md
M asterixdb/asterix-lang-aql/src/main/javacc/AQL.jj
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/expression/OperatorExpr.java
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/struct/OperatorType.java
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/util/CommonFunctionMapUtil.java
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/BuiltinFunctions.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/NumericDivideTypeComputer.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/AbstractNumericArithmeticEval.java
C 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericDivDescriptor.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericDivideDescriptor.java
R 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericPowerDescriptor.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/functions/FunctionCollecti

Change in asterixdb[master]: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2378][SQL] Add “DIV”, “MOD”, change “/“ for integers
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/4191/ (7/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c6b0704ce60a03dd3c10e1c75cb9761acc56536
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5: Contrib+1

BAD Compatibility Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Analytics Compatibility Compilation Successful
https://goo.gl/fN5rjL : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/3689/ 
(4/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/3462/ 
(9/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/3474/
 (2/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/3386/ 
(7/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/3905/
 (5/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/7413/ (3/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Michael Blow (Code Review)
Michael Blow has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5: Contrib+1 Integration-Tests+1 Verified+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/4190/ (8/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/1481/ 
(12/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/3433/ 
(11/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/3926/ 
(6/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/8937/ (1/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/4006/ (10/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [ASTERIXDB-2370][*DB] Modify the inconsistent method name "i...

2018-05-01 Thread Michael Blow (Code Review)
Hello Anon. E. Moose #1000171, Jenkins,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#5).

Change subject: [ASTERIXDB-2370][*DB] Modify the inconsistent method name 
"isFull"
..

[ASTERIXDB-2370][*DB] Modify the inconsistent method name "isFull"

- taken from https://github.com/apache/asterixdb/pull/7

The method is named as "isFull". However, the method does set the boolean
variable full.  Thus, rename the method as "setFull" should be more
intuitive than "isFull" since "isFull" is a query asking whether the 
object is full or not.

Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
---
M 
asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/logging/ReplicationLogBuffer.java
M 
asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/management/LogReplicationManager.java
2 files changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/29/2629/5
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2629
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 


[jira] [Created] (ASTERIXDB-2378) Add DIV, MOD, change handling of integers operands in /

2018-05-01 Thread Dmitry Lychagin (JIRA)
Dmitry Lychagin created ASTERIXDB-2378:
--

 Summary: Add DIV, MOD, change handling of integers operands in /
 Key: ASTERIXDB-2378
 URL: https://issues.apache.org/jira/browse/ASTERIXDB-2378
 Project: Apache AsterixDB
  Issue Type: Improvement
  Components: SQL - Translator SQL++
Reporter: Dmitry Lychagin
Assignee: Dmitry Lychagin


1) Change  "/" operator to return double if both operands are integers

2) Add "DIV" operator which is the same as "/" except it returns integer if 
both operands are integers

3) Add "MOD" operator which is an alias for "%" operator

4) Remove "IDIV" operator from the grammar



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4: Contrib+1

Analytics Compatibility Tests Successful
https://goo.gl/QyRjf8 : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4: Contrib+1

BAD Compatibility Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Analytics Compatibility Compilation Successful
https://goo.gl/qLLL53 : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/3385/ 
(12/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/3902/
 (4/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/3430/ 
(9/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/1478/ 
(11/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/8934/ (1/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 3:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 3:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 3: Contrib-2

Analytics Compatibility Compilation Failed
https://goo.gl/kXLWCE : UNSTABLE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 2: Contrib-2

Analytics Compatibility Compilation Failed
https://goo.gl/NTJNZ6 : UNSTABLE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/7412/ (2/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/8936/ (1/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/3473/
 (7/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/3688/ 
(9/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/3925/ 
(8/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/4005/ (6/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/4189/ (3/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-2370: Modify the inconsistent method name "isFull".

2018-05-01 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-2370: Modify the inconsistent method name "isFull".
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/1480/ 
(11/12)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bf451367171afb3fc0511a6170eb4c492fc69f
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


  1   2   >