Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
abdullah alamoudi has submitted this change and it was merged.
Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push
back
..
[NO ISSUE][LOG] Include destination IP in thread name on push back
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- In certain cases, we noticed that threads of certain jobs are
waiting on the network boundaries to send tuples.
- In that case and in a big cluster, it is difficult to tell which
node is pushing back.
- To make this easy, we include the IP of the receiving node
in the thread name.
Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2929
Sonar-Qube: Jenkins
Tested-by: Jenkins
Contrib: Jenkins
Reviewed-by: Murtadha Hubail
Integration-Tests: Jenkins
---
M
hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
M
hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
M
hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
M
hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/MultiplexedConnection.java
4 files changed, 25 insertions(+), 1 deletion(-)
Approvals:
Anon. E. Moose #1000171:
Jenkins: Verified; No violations found; ; Verified
Murtadha Hubail: Looks good to me, approved
diff --git
a/hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
b/hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
index 334fb5c..261e7c4 100644
---
a/hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
+++
b/hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
@@ -18,9 +18,11 @@
*/
package org.apache.hyracks.comm.channels;
+import java.net.InetSocketAddress;
import java.nio.ByteBuffer;
import java.util.ArrayDeque;
import java.util.Deque;
+import java.util.Objects;
import org.apache.hyracks.api.comm.IBufferAcceptor;
import org.apache.hyracks.api.comm.IFrameWriter;
@@ -75,7 +77,15 @@
break;
}
try {
-wait();
+InetSocketAddress remoteAddress =
ccb.getRemoteAddress();
+String nameBefore = Thread.currentThread().getName();
+try {
+Thread.currentThread()
+.setName(nameBefore + ":SendingTo(" +
Objects.toString(remoteAddress) + ")");
+wait();
+} finally {
+Thread.currentThread().setName(nameBefore);
+}
} catch (InterruptedException e) {
Thread.currentThread().interrupt();
throw HyracksDataException.create(e);
diff --git
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
index 998acfb..f7ef2aa 100644
---
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
+++
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
@@ -19,6 +19,7 @@
package org.apache.hyracks.net.protocols.muxdemux;
import java.io.IOException;
+import java.net.InetSocketAddress;
import java.nio.channels.SocketChannel;
import java.util.concurrent.atomic.AtomicBoolean;
@@ -163,4 +164,8 @@
+ " remoteClose: " + remoteClose + " remoteCloseAck:" +
remoteCloseAck + " readCredits: "
+ ri.getCredits() + " writeCredits: " + wi.getCredits() + "]";
}
+
+public InetSocketAddress getRemoteAddress() {
+return cSet.getMultiplexedConnection().getRemoteAddress();
+}
}
diff --git
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
index 31a37ef..f5cdf2c 100644
---
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
+++
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
@@ -239,4 +239,8 @@
ccbArray = Arrays.copyOf(ccbArray, ccbArray.length * 2);
}
}
+
+public
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/7240/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: -Integration-Tests Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/7240/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Murtadha Hubail has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Integration-Tests-1 Integration Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/7236/ : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Contrib+1 Analytics Compatibility Tests Successful https://goo.gl/8wu6RJ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Contrib+1 BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/3629/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/3629/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/4765/ (13/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Analytics Compatibility Compilation Successful https://goo.gl/zFq7CP : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/4323/ (12/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/7236/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/4413/ (7/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage-jre10/683/ (4/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/8350/ (10/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/4378/ (6/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/4970/ (1/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/4322/ (12/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/4802/ (2/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/9889/ (5/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-java10/608/ (3/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/4412/ (9/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/4632/ (11/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/2424/ (8/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Analytics Compatibility Compilation Successful https://goo.gl/QM2We6 : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
abdullah alamoudi has uploaded a new patch set (#2). Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. [NO ISSUE][LOG] Include destination IP in thread name on push back - user model changes: no - storage format changes: no - interface changes: no Details: - In certain cases, we noticed that threads of certain jobs are waiting on the network boundaries to send tuples. - In that case and in a big cluster, it is difficult to tell which node is pushing back. - To make this easy, we include the IP of the receiving node in the thread name. Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d --- M hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java M hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java M hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java M hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/MultiplexedConnection.java 4 files changed, 25 insertions(+), 1 deletion(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/29/2929/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/7235/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
abdullah alamoudi has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/2929
Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push
back
..
[NO ISSUE][LOG] Include destination IP in thread name on push back
Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d
---
M
hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
M
hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
M
hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
M
hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/MultiplexedConnection.java
4 files changed, 25 insertions(+), 1 deletion(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/29/2929/1
diff --git
a/hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
b/hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
index 334fb5c..261e7c4 100644
---
a/hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
+++
b/hyracks-fullstack/hyracks/hyracks-comm/src/main/java/org/apache/hyracks/comm/channels/NetworkOutputChannel.java
@@ -18,9 +18,11 @@
*/
package org.apache.hyracks.comm.channels;
+import java.net.InetSocketAddress;
import java.nio.ByteBuffer;
import java.util.ArrayDeque;
import java.util.Deque;
+import java.util.Objects;
import org.apache.hyracks.api.comm.IBufferAcceptor;
import org.apache.hyracks.api.comm.IFrameWriter;
@@ -75,7 +77,15 @@
break;
}
try {
-wait();
+InetSocketAddress remoteAddress =
ccb.getRemoteAddress();
+String nameBefore = Thread.currentThread().getName();
+try {
+Thread.currentThread()
+.setName(nameBefore + ":SendingTo(" +
Objects.toString(remoteAddress) + ")");
+wait();
+} finally {
+Thread.currentThread().setName(nameBefore);
+}
} catch (InterruptedException e) {
Thread.currentThread().interrupt();
throw HyracksDataException.create(e);
diff --git
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
index 998acfb..f7ef2aa 100644
---
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
+++
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
@@ -19,6 +19,7 @@
package org.apache.hyracks.net.protocols.muxdemux;
import java.io.IOException;
+import java.net.InetSocketAddress;
import java.nio.channels.SocketChannel;
import java.util.concurrent.atomic.AtomicBoolean;
@@ -163,4 +164,8 @@
+ " remoteClose: " + remoteClose + " remoteCloseAck:" +
remoteCloseAck + " readCredits: "
+ ri.getCredits() + " writeCredits: " + wi.getCredits() + "]";
}
+
+public InetSocketAddress getRemoteAddress() {
+return cSet.getMultiplexedConnection().getRemoteAddress();
+}
}
diff --git
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
index 31a37ef..f5cdf2c 100644
---
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
+++
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelSet.java
@@ -239,4 +239,8 @@
ccbArray = Arrays.copyOf(ccbArray, ccbArray.length * 2);
}
}
+
+public MultiplexedConnection getMultiplexedConnection() {
+return mConn;
+}
}
diff --git
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/MultiplexedConnection.java
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/MultiplexedConnection.java
index b6a392e..4c3836a 100644
---
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/MultiplexedConnection.java
+++
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/4801/ (9/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-java10/607/ (3/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/2423/ (11/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/4969/ (1/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/4377/ (7/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/9888/ (5/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage-jre10/682/ (6/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/4412/ (8/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/8349/ (4/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/4631/ (10/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][LOG] Include destination IP in thread name on pus...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][LOG] Include destination IP in thread name on push back .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/4411/ (2/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/2929 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I81b48cb7a79de0a37f2171a1bd3b82141f89b49d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
