Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Anon. E. Moose (Code Review) 
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2: Contrib+1

Analytics Compatibility Tests Successful Build 2580
: SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2: Contrib+1

BAD Compatibility Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/1809/ (10/10)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/1624/ 
(9/10)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/6446/ (5/10)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/890/ (1/10)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Luo Chen (Code Review) 
Hello Jenkins,

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

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

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

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..

[ASTERIXDB-2090] Fix componentIds for bulk load

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

Details:
Currently we use LSN as component Ids during the flush time. However,
bulk loaded components receive no component Ids because of no
transactions on them. This patch is a temp fix for this bug by simply
giving bulk loaded components with id 0. This is correct since
currently we only allow load an empty dataset.

An alternative (and better) approach is to use the timestamp instead of
LSNs as component Ids. This needs some discussions, and might be
implemented later.

Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
---
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/ioopcallbacks/AbstractLSMIOOperationCallback.java
1 file changed, 0 insertions(+), 8 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/15/2015/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2015
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Ildar Absalyamov 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1228/ 
(3/10)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/6444/ (8/10)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/893/
 (4/10)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/893/ 
(5/10)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2090] Fix componentIds for bulk load

2017-09-17 Thread Luo Chen (Code Review) 
Luo Chen has uploaded a new change for review.

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

Change subject: [ASTERIXDB-2090] Fix componentIds for bulk load
..

[ASTERIXDB-2090] Fix componentIds for bulk load

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

Details:
Currently we use LSN as component Ids during the flush time. However,
bulk loaded components receive no component Ids because of no
transactions on them. This patch is a temp fix for this bug by simply
giving bulk loaded components with id 0. This is correct since
currently we only allow load an empty dataset.

An alternative (and better) approach is to use the timestamp instead of
LSNs as component Ids. This needs some discussions, and might be
implemented later.

Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
---
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/ioopcallbacks/AbstractLSMIOOperationCallback.java
1 file changed, 0 insertions(+), 5 deletions(-)


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

diff --git 
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/ioopcallbacks/AbstractLSMIOOperationCallback.java
 
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/ioopcallbacks/AbstractLSMIOOperationCallback.java
index f357aea..5806437 100644
--- 
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/ioopcallbacks/AbstractLSMIOOperationCallback.java
+++ 
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/ioopcallbacks/AbstractLSMIOOperationCallback.java
@@ -20,7 +20,6 @@
 package org.apache.asterix.common.ioopcallbacks;
 
 import java.util.List;
-import java.util.logging.Level;
 import java.util.logging.Logger;
 
 import org.apache.hyracks.api.exceptions.HyracksDataException;
@@ -117,10 +116,6 @@
 //if oldComponents == null, then getComponentLSN would treat it as 
a flush operation,
 //and return the LSN for the flushed component
 long id = getComponentLSN(null);
-if (id == 0) {
-logger.log(Level.WARNING, "Flushing a memory component without 
setting the LSN");
-id = ILSMDiskComponentId.NOT_FOUND;
-}
 return new LSMDiskComponentId(id, id);
 } else {
 long minId = Long.MAX_VALUE;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen