Versions Compared

Old Version 5

changes.mady.by.user Kirk Lund

Saved on

compared with

New Version Current

changes.mady.by.user Dan Smith

Saved on

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

GEODE-nn: Capitalized, 50 chars or less summary

More detailed explanation with linefeeds to wrap at 72 characters after a blank line following the summary.

...

  • GEODE-nn: Fixing failing CompositePropertySourceTest
  • GEODE-nn: Fixed failing CompositePropertySourceTest
  • GEODE-nn: failing CompositePropertySourceTest
  • GEODE-nn: polishing stuff
  • GEODE-nn: CompositePropertySourceTest is failing intermittently
  • GEODE-nn: variable name fixes

[1] How to write a Git commit message.


Examples


Here's an example of a good commit message:

No Format

GEODE-9666: Avoid caching InetSocketAddress (#6938)

The changes for GEODE-9139 changed the behavior of
org.apache.geode.distributed.internal.tcpserver.HostAndPort to
permanently cache the internal InetSocketAddress once it has tried one
time to resolve the address. This undoes part of the fix introduced by
GEODE-7808, in which HostAndPort was created as a way to hold an
unresolved hostname.

The issue is that the cached InetSocketAddress may contain a stale or
unresolved address which will be returned by getSocketInetAddress for
the lifetime of the HostAndPort/InetSocketWrapper object. This prevents
the address from being resolved correctly after changes in DNS records.
(Such changes are common in cloud environments.)

This commit removes the cached internal InetSocketAddress from
InetSocketWrapper so that getSocketInetAddress will try to resolve the
address each time it is called with an unresolved address