MAPREDUCE-7156. NullPointerException when reaching max shuffle connections. Contributed by Peter Bacsko

This commit is contained in:
Jason Lowe 2018-11-06 17:55:51 -06:00
parent 08d69d91f2
commit ba1f9d66d9

View File

@ -910,6 +910,8 @@ private List<String> splitMaps(List<String> mapq) {
@Override @Override
public void channelOpen(ChannelHandlerContext ctx, ChannelStateEvent evt) public void channelOpen(ChannelHandlerContext ctx, ChannelStateEvent evt)
throws Exception { throws Exception {
super.channelOpen(ctx, evt);
if ((maxShuffleConnections > 0) && (accepted.size() >= maxShuffleConnections)) { if ((maxShuffleConnections > 0) && (accepted.size() >= maxShuffleConnections)) {
LOG.info(String.format("Current number of shuffle connections (%d) is " + LOG.info(String.format("Current number of shuffle connections (%d) is " +
"greater than or equal to the max allowed shuffle connections (%d)", "greater than or equal to the max allowed shuffle connections (%d)",
@ -925,8 +927,6 @@ public void channelOpen(ChannelHandlerContext ctx, ChannelStateEvent evt)
return; return;
} }
accepted.add(evt.getChannel()); accepted.add(evt.getChannel());
super.channelOpen(ctx, evt);
} }
@Override @Override