• Tom Lane's avatar
    De-support floating-point timestamps. · b6aa17e0
    Tom Lane authored
    Per discussion, the time has come to do this.  The handwriting has been
    on the wall at least since 9.0 that this would happen someday, whenever
    it got to be too much of a burden to support the float-timestamp option.
    The triggering factor now is the discovery that there are multiple bugs
    in the code that attempts to implement use of integer timestamps in the
    replication protocol even when the server is built for float timestamps.
    The internal float timestamps leak into the protocol fields in places.
    While we could fix the identified bugs, there's a very high risk of
    introducing more.  Trying to build a wall that would positively prevent
    mixing integer and float timestamps is more complexity than we want to
    undertake to maintain a long-deprecated option.  The fact that these
    bugs weren't found through testing also indicates a lack of interest
    in float timestamps.
    
    This commit disables configure's --disable-integer-datetimes switch
    (it'll still accept --enable-integer-datetimes, though), removes direct
    references to USE_INTEGER_DATETIMES, and removes discussion of float
    timestamps from the user documentation.  A considerable amount of code is
    rendered dead by this, but removing that will occur as separate mop-up.
    
    Discussion: https://postgr.es/m/26788.1487455319@sss.pgh.pa.us
    b6aa17e0
ecpg_config.h.in 454 Bytes