Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-18 Thread Murtadha Hubail (Code Review) 
Murtadha Hubail has abandoned this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Abandoned

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

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

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1: Contrib+1

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

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

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

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

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

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1: Contrib-2

Analytics Compatibility Tests Failed
https://goo.gl/R3UHuf : UNSTABLE

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

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

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

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

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

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

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1: Contrib+1

BAD Compatibility Tests Successful

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

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

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

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

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

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/160/
 (13/13)

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

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

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

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

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

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

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/158/
 (12/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Murtadha Hubail (Code Review) 
Murtadha Hubail has uploaded a new change for review.

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

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..

[ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing It

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

Details:
- When recycling a buffer, calculate the buffer remaining before
  releasing it for reuse to prevent other threads from changing
  its remaining.

Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
---
M 
hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/FullFrameChannelReadInterface.java
1 file changed, 1 insertion(+), 1 deletion(-)


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

diff --git 
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/FullFrameChannelReadInterface.java
 
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/FullFrameChannelReadInterface.java
index 32bf77e..054acb6 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/FullFrameChannelReadInterface.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/FullFrameChannelReadInterface.java
@@ -45,8 +45,8 @@
 if (ccb.isRemotelyClosed()) {
 return;
 }
-riEmptyStack.push(buffer);
 final int delta = buffer.remaining();
+riEmptyStack.push(buffer);
 ccb.addPendingCredits(delta);
 };
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/4694/ 
(7/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/4949/ 
(6/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/8676/ (4/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reus...

2018-11-14 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2478][NET] Calculate Buffer Remaining Before Reusing 
It
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/2738/ 
(5/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I15629ec57c42944c24815e923672fb0f9c34e278
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No