First off, you don't need to use the statement With Me because it's redundant.

Second, in an If statement there's only need for (in my oppinion) one Else.
As the If statement is for checking...

I would try to limit the amount of data for each query.
Using the reader in order to populate some kind of repository do take a long time.
The way the reader works is that it only reads one line of...

If the database was to be openly accessible *shudder*, albeit with a username and password.
Then you can most likely access the database with javascript and odbc (jdbc?).
If that is the case then...