[GitHub] trafficserver pull request: Maintain and use a mapping of hostname...

2015-07-01 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/240#issuecomment-117899443 I reviewed this and iterated with @jacksontj and so obviously I give this a +1, @SolidWallOfCode can you please take a look at this latest version? --- If your pro

[GitHub] trafficserver pull request: Maintain and use a mapping of hostname...

2015-07-01 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/240#issuecomment-117899355 @SolidWallOfCode This should fix your hosts file feature (TS-3725) --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver pull request: Maintain and use a mapping of hostname...

2015-07-01 Thread jacksontj
GitHub user jacksontj opened a pull request: https://github.com/apache/trafficserver/pull/240 Maintain and use a mapping of hostname -> IP to implement host file support This allows the background thread to populate a map of overrides, which we simply inject in the lookup method-- i

[GitHub] trafficserver pull request: Revert "TS-3237: Don't segregate DNS r...

2015-07-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/238 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] trafficserver pull request: TS-3718 Remove unused member variables...

2015-07-01 Thread danobi
GitHub user danobi opened a pull request: https://github.com/apache/trafficserver/pull/239 TS-3718 Remove unused member variables/functions read_metadata() and m_size_bytes are not used anywhere in the code base. You can merge this pull request into a Git repository by running:

[GitHub] trafficserver pull request: Revert "TS-3237: Don't segregate DNS r...

2015-07-01 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/238#issuecomment-117792235 @zwoop you said to run clang-format, but when I do there are LOTS of changes to this file-- so I assume we aren't using the default standard clang picks? I sear

[GitHub] trafficserver pull request: Revert "TS-3237: Don't segregate DNS r...

2015-07-01 Thread jacksontj
GitHub user jacksontj opened a pull request: https://github.com/apache/trafficserver/pull/238 Revert "TS-3237: Don't segregate DNS results by port." Fixes: TS-3720 You can merge this pull request into a Git repository by running: $ git pull https://github.com/jacksontj/traffics