<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">So my only concern is that 13 and 16 are LTS releases and we probably shouldn't break support for older (even EOL) versions of PostgreSQL.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'd argue that even if we tried to support that old PostgreSQL versions, we'd have a hard time testing that, as there could be significant effort involved (make old postgresql compile on new OS, find old OS, ...) to even get a server that old.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">All 3 of the affected modules call PQserverVersion() at some point to determine the version of the server. Could we do the same here to ensure backwards compatibility?</p></blockquote><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Let's see: we don't have to care about servers "7.3 and older", as they already have their own SQL statement and I didn't touch that. Digging the old docs, I see that version 7.4 already has pg_attrdef.adbin and pg_get_expr():<br> - https://www.postgresql.org/docs/7.4/catalog-pg-attrdef.html<br> - https://www.postgresql.org/docs/7.4/functions-misc.html#FUNCTIONS-MISC-CATALOG-TABLE<br>so at least in theory, the code should work. (The documentation of function pg_get_expr() has been moved around - it's now in "System Information Functions" https://www.postgresql.org/docs/12/functions-info.html#FUNCTIONS-INFO-CATALOG-TABLE and the automatic references across PostgreSQL releases doesn't work in that case). My initial assumption that the code wouldn't work seems to be wrong (just didn't find the relevant docs), but for testing that... see above.</pre></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I think based on your research it seems safe within the range of versions the change is used against.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/13023">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/13023">change 13023</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/13023"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-Change-Id: I834cb3addf1937e19e87ede140bdd16cea531ebe </div>
<div style="display:none"> Gerrit-Change-Number: 13023 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Christoph Moench-Tegeder <cmt@burggraben.net> </div>
<div style="display:none"> Gerrit-Reviewer: Christoph Moench-Tegeder <cmt@burggraben.net> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-CC: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-CC: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 09 Oct 2019 13:10:13 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>