Github user jacksontj commented on the pull request:
https://github.com/apache/trafficserver/pull/526#issuecomment-196921878
As I'm thinking through this some more, I'm not certain this is a great
solution either. The particular case that I reproduced was that hostdb was full
when returning rr() records. It seems that the `insert` into hostdb can't fail
(as it will eject things if necessary), so really the core issue here is that
we are trying to store some additional information (hostnames, etc.) and we
cannot store them because we are out of space. The current patch I have here is
just returning the HostDBInfo struct, but it didn't put anything in where the
hostnames should be. In the more simple cases (where we are storing the
hostname that we looked up) it is probably preferable to return without a
hostname at all, but in the reverse DNS case that is the primary intent of the
lookup.
Thoughts?
---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
