Clone Tools
  • last updated 11 mins ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch 'gerrit/stabilization-f69489'

Change-Id: I5b75e79a5dbe74314fec8b9ef7734a792ca5731c

    • -2
    • +4
    ./net/protocols/muxdemux/ChannelSet.java
  1. … 4 more files in changeset.
[NO ISSUE][NET] Ensure Channel Was Not Recycled Before Adding Credit

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Before attempting to add credit to an existing channel,

ensure the channel slot was not recycled and reused by a

new channel. Without this, some additional credit might

erroneously be added to the new channel.

Change-Id: I0a5ec70ad4c2568477fac9eb98c030fc3d8b1ce9

Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/4623

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

Contrib: Michael Blow <mblow@apache.org>

    • -2
    • +4
    ./net/protocols/muxdemux/ChannelSet.java
[NO ISSUE][NET] Exclude Warn Logs When No Buffer Factory is Used

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Do not warn about recycled partial frames when no buffer

factory is set in FullFrameChannelReadInterface as those

are valid cases that use a single frame to perform the

complete operation.

Change-Id: I80251c529eb15ee22d01c320883418ea80272c2c

Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/3824

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Till Westmann <tillw@apache.org>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

[NO ISSUE][NET] Log ChannelReadInterface Stats on Failure

- user model changes: no

- storage format changes: no

- interface changes: yes

Details:

- When the read buffers of a FullFrameChannelReadInterface

are exceeded, log its stats to help in debugging the issue.

- Warn when a partial frame is being written over a full

frame write channel.

- Rename ThreadSafetyGuaranteedBy -> GuardedBy.

- Annotate networking calls with their synchronization

guards.

Change-Id: I89eed0c06dbf4b0e86747538bff286dc37853957

Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/3663

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Al Hubail <mhubail@uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

  1. … 7 more files in changeset.
Merge commit '8930f0d' from stabilization-f69489

Change-Id: I64cf862bc5ebe19a24acbccf821f68556699222e

  1. … 23 more files in changeset.
[NO ISSUE][NET] Ensure Recycling Buffer and Notifying Sender is Atomic

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- To avoid synchronization issues that might occur

due to JVM reordering, ensure that both recycling

read buffers and notifying the sender of their

availability are done atomically before the next

buffer is received from the sender.

Change-Id: Ia3b1920f33bf7d4e7efbd2ea3405cbc4310a78c7

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3520

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Till Westmann <tillw@apache.org>

(cherry picked from commit 32eed5f384c5851eae1c613fcb3b9532744ed595)

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3525

Reviewed-by: Michael Blow <mblow@apache.org>

[NO ISSUE][NET] Ensure Handshake Connections List is Thread-Safe

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Ensure the list of connections that completed their asynchronous

SSL handshake is accessed in thread-safe manner.

Change-Id: Ibfd4fc569b59f39d730ef04f5d69b23d91713ed7

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3523

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

    • -10
    • +16
    ./net/protocols/tcp/TCPEndpoint.java
[NO ISSUE][NET] Ensure Recycling Buffer and Notifying Sender is Atomic

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- To avoid synchronization issues that might occur

due to JVM reordering, ensure that both recycling

read buffers and notifying the sender of their

availability are done atomically before the next

buffer is received from the sender.

Change-Id: Ia3b1920f33bf7d4e7efbd2ea3405cbc4310a78c7

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3520

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Till Westmann <tillw@apache.org>

Merge commit '8c99391' from stabilization-f69489

Change-Id: I139c265ba998c32ec049c8a8bbd7a5a213895d2a

  1. … 7 more files in changeset.
[NO ISSUE][NET] Do Not Flush Partial Frames on Network Errors

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Do not flush partially read frames when a network error

is reported to avoid processing invalid partial frames

before the network failure is detected by the channel

reader.

Change-Id: Ia35b58412cc293426c18a7cd66d8d0a5962db4e2

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3503

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Till Westmann <tillw@apache.org>

[NO ISSUE] Apply / enforce java import order

The process-sources target will now sort imports as well as

format source code; the source-format job will likewise verify

import order in addition to source code format

Change-Id: I55d976c4df10d9919c6a25683be2a3e3304e65d9

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3288

Integration-Tests: Michael Blow <mblow@apache.org>

Tested-by: Michael Blow <mblow@apache.org>

Reviewed-by: Till Westmann <tillw@apache.org>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -1
    • +1
    ./net/protocols/muxdemux/MuxDemux.java
  1. … 623 more files in changeset.
[ASTERIXDB-2490][NET] Support Encrypted Multiplexed Connections

- user model changes: no

- storage format changes: no

- interface changes: yes

Details:

- Use SocketChannelFactory in multiplex connections

to support both unencrypted and encrypted sockets.

- Adapt TCPEndpoint to socket channels that require

handshake.

- Adapt test cases to API changes.

Change-Id: I9cbed93c162018bad17923d50d4987011cbba16c

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3071

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Michael Blow <mblow@apache.org>

    • -6
    • +18
    ./net/protocols/muxdemux/MultiplexedConnection.java
    • -3
    • +9
    ./net/protocols/muxdemux/MuxDemux.java
    • -6
    • +6
    ./net/protocols/tcp/TCPConnection.java
    • -18
    • +115
    ./net/protocols/tcp/TCPEndpoint.java
  1. … 12 more files in changeset.
[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.

- Add test case.

Change-Id: Icca3284feae800dd6c37694bdefec3516cd4c506

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3036

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

  1. … 2 more files in changeset.
[ASTERIXDB-2479][API] Add Network Diagnostics API

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Add a new API that shows the state of node to node

connections and their logical channels to help diagnose

networking issues.

- Add channel details to the waiting thread name in

NetworkOutputChannel.

- Add test case.

Change-Id: Id6fd5a96c56e7078d1404bebcbab8afe93ba8f64

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3025

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Michael Blow <mblow@apache.org>

    • -0
    • +13
    ./net/protocols/muxdemux/ChannelSet.java
    • -0
    • +17
    ./net/protocols/muxdemux/MultiplexedConnection.java
    • -8
    • +34
    ./net/protocols/muxdemux/MuxDemux.java
  1. … 11 more files in changeset.
[NO ISSUE][NET] Ensure CLOSE Is Not Sent After Channel ERROR

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Currently it is possible to send network channel

CLOSE command after a channel ERROR was sent. When this

happens and the channel was recycled to be reused

on the receiver side, the CLOSE command will result

in NPE. There is no need to send a CLOSE command

after an ERROR command because when an ERROR command

is received, it is treated as ERROR + CLOSE on the

receiver side.

- Avoid registering partition requests for failed jobs.

Change-Id: I17a769a46f4d13220adb22dd255e56dc4ccc458d

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2954

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Michael Blow <mblow@apache.org>

  1. … 2 more files in changeset.
Revert "[NO ISSUE][LOG] Include destination IP in thread name on push back"

This reverts commit 2498d28f0584d6ab002625417622be429f5c8131.

Change-Id: I221071a8a2851bde839e67d7f0380816c249366d

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2941

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

    • -4
    • +0
    ./net/protocols/muxdemux/ChannelSet.java
  1. … 1 more file in changeset.
[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 <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

(cherry picked from commit 2498d28f0584d6ab002625417622be429f5c8131)

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2942

    • -0
    • +4
    ./net/protocols/muxdemux/ChannelSet.java
  1. … 1 more file in changeset.
[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 <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -0
    • +4
    ./net/protocols/muxdemux/ChannelSet.java
  1. … 1 more file in changeset.
[NO ISSUE][NET] Ensure Thread Safety in FullFrameChannelReadInterface

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Use a blocking deque in FullFrameChannelReadInterface

to ensure thread safety between frame consumer and

the networking thread.

Change-Id: I33f0171e49b0ff972730a678e8b61a2070dc8832

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2921

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Michael Blow <mblow@apache.org>

[NO ISSUE][NET] Log Channel Info When Buffers Are Exceeded

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- When the read buffers of a network channel are

exceeded, log the channel information and throw

an illegal state exception.

Change-Id: I5290947f57c91d4a537330eef91cf23d9352f24c

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2917

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Michael Blow <mblow@apache.org>

[NO ISSUE][NET] Skip Channel Write on Connection Failure

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Do not attempt to adjust channel writability if

the failure was due to connection failure since

no more messages will be sent on that connection.

This is done to prevent a possible deadlock between

network IOThread that detected connection failure

and another thread that might be accessing the channel.

- Make sending error code conditions more explicit since

we currently have a single error code that is sent.

Change-Id: Ic25f05ac2c0d02699324f2d1b80c51f392654106

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2892

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -1
    • +1
    ./net/protocols/muxdemux/ChannelSet.java
  1. … 3 more files in changeset.
[NO ISSUE][OTH] Move Network Tracing Logs to Trace Level

- user model changes: no

- storage format changes: no

- interface changes: no

Change-Id: Iebc13ff6cb7b3c9086c19597ec982061672a891f

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2848

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: abdullah alamoudi <bamousaa@gmail.com>

    • -4
    • +4
    ./net/protocols/muxdemux/ChannelSet.java
  1. … 8 more files in changeset.
[NO ISSUE][NET] Remove Unneeded Sync From ChannelControlBlock

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

Remove synchronization from ChannelControlBlock

reportRemoteError since the caller will either be

synchronized on the multiplex connection or it will

be a single network IOThread notifying the channel.

This is done to prevent a possible deadlock between

threads detecting multiplex connection failure and

pipeline failing threads.

Change-Id: Ic81946ffea7fcb28ec1d96eae86d2473bdc5aef2

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2828

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: abdullah alamoudi <bamousaa@gmail.com>

[NO ISSUE][RT] Abort tasks on local network failures

- user model changes: no

- storage format changes: no

- interface changes: yes

Add error code to IInputChannelMonitor.notifyFailure

Details:

- Previously, there was an assumption that all failures

reported to an IInputChannelMonitor come from a remote

task.

- This assumption is not always true and could lead

to jobs hanging.

- To fix this, we report an error code indicating whether

the failure is local or remote and if the failure is local

then we fail the local task and report the failure to cc.

Change-Id: I7ea5b9008383faaac7c563671242b03919090b35

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2806

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -1
    • +1
    ./net/protocols/muxdemux/ChannelSet.java
  1. … 12 more files in changeset.
[NO ISSUE][NET] Abort All Channels on Network Failures

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Previously network output channels were excluded

from the channels to be notified of failures and

aborted since their remote stream is usually closed.

- This change ensures that all channels are notified of

failures since the channels TCP connection is closed

on network IO errors.

Change-Id: I983be62f314401aa5ee2ef3b4f02cb23ce5f6de7

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2807

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: abdullah alamoudi <bamousaa@gmail.com>

    • -1
    • +1
    ./net/protocols/muxdemux/ChannelSet.java
[NO ISSUE][NET] Networking Fixes

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Ensure received partitions requests after job failure

are aborted to prevent leaked network channels.

- Do not send channel close after channel write error

since the contract is to close the channel when remote

errors are received.

- Only remove closed outgoing connections to establish

new connections since incoming connections need to be

reestablished by the remote destination.

- Do not perform further operations on failed multiplexed

connections to avoid CanceledKeyException.

- Add test case for received partition requests after

job failure.

Change-Id: Idc45f47fdf0419bf75d461e16f028237a5143de7

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2613

Reviewed-by: Michael Blow <mblow@apache.org>

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Michael Blow <mblow@apache.org>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -22
    • +30
    ./net/protocols/muxdemux/MultiplexedConnection.java
    • -1
    • +3
    ./net/protocols/muxdemux/MuxDemux.java
    • -1
    • +14
    ./net/protocols/tcp/TCPConnection.java
  1. … 7 more files in changeset.
[NO ISSUE][NET] Improve Logging in TCPEndpoint

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Include end point local address in IO thread name.

- Log TCPEndpoint/Connection addresses on failures.

- Fix multiple IO Threads initialization.

- Fix synchronization for closed connection notification.

Change-Id: I5e78755150a6424fd22f587c4311da7c60b3b55a

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2487

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

    • -0
    • +5
    ./net/protocols/tcp/TCPConnection.java
    • -10
    • +17
    ./net/protocols/tcp/TCPEndpoint.java
[NO ISSUE][NET] Networking Improvements

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Set keep alive and no TCP delay options

on socket channels.

- Cancel key and close IPC handle on failed

read/writes to avoid getting the same failures

with every NetworkThread loop.

Change-Id: I60c1f9cfe2ea577fca14cd2e98c6461c49df011a

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2418

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

  1. … 6 more files in changeset.
[ASTERIXDB-2256] Reformat sources using code format template

Change-Id: I4faa141c1a8c9700d5e9ac50b839acc9d1eede73

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2310

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

  1. … 984 more files in changeset.
[ASTERIXDB-2236][NET] Reestablish Nc2Nc Closed Connection

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Enable keep alive on Nc2Nc sockets.

- Open Nc2Nc connection when closed.

- Add "port" command to test framework to

ensure a port is not used anymore.

- Add test case for reestablishing connection

between nodes.

Change-Id: I3af4af37de6c1b36d7032c78ee60f1e5ca5f7c1c

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2273

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

    • -0
    • +7
    ./net/protocols/muxdemux/MuxDemux.java
    • -4
    • +11
    ./net/protocols/tcp/TCPConnection.java
    • -5
    • +15
    ./net/protocols/tcp/TCPEndpoint.java
  1. … 19 more files in changeset.