Skip to content

Convert socket adaptor to socket #160

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Suraiya-Hameed merged 2 commits into from
Feb 28, 2017
Merged

Convert socket adaptor to socket #160

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d346181
create a java.net.Socket object instead of SocketAdaptor
xiangyushawn Feb 25, 2017
c086691
close the previous SocketChannel
xiangyushawn Feb 27, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 10 additions & 5 deletions src/main/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2552,13 +2552,18 @@ private void findSocketUsingJavaNIO(InetAddress[] inetAddrs,
}
}

// if a channel was selected, make the necessary updates
// if a channel was selected, make the necessary updates
if (selectedChannel != null) {
// Note that this must be done after selector is closed. Otherwise,
// we would get an illegalBlockingMode exception at run time.
selectedChannel.configureBlocking(true);
selectedSocket = selectedChannel.socket();
//the selectedChannel has the address that is connected successfully
//convert it to a java.net.Socket object with the address
SocketAddress iadd = selectedChannel.getRemoteAddress();
selectedSocket = new Socket();
selectedSocket.connect(iadd);

result = Result.SUCCESS;

//close the channel since it is not used anymore
selectedChannel.close();
}
}

Expand Down