2014-11-10 17:42 GMT+01:00 Mark Thomas <ma...@apache.org>: > Hi, > > I am about to push the first batch of commits. I started with the > upgrade package as that looked like the simplest to tackle first. Rather > than a single commit, I'll keep the individual commits from git. They > aren't all as clean as I would like but it gives a better idea of how I > approached it and (I think) is easier to review. > > The unit tests all pass for all three connectors on OSX. >
Ok, nice. But I'm a bit cautions, since some flexibility is lost. For example, with the NIO2 connector, the code is rather different between upgrade and non upgrade, plus it has the possibility of doing gather writes (not done in upgrade, only in regular mode; basically in upgrade mode the framework in place was inflexible, so I didn't use gather for the NIO2 connector). So if the idea is to merge everything in a single read/write method in the wrapper, then it's not a good idea. Ultimately being able to do splitter/gather in upgrade mode should be good (if needed). > > My thinking moving forward is to do AJP next and HTTP last. At some > point I may move the SocketWrappers to separate classes as they are > fairly large. > > AJP ? I don't see it working any better than the java.io connector for the new technologies, so the plan should probably be to remove it or have it limited to legacy mode. Rémy
