Github user kaknikhil commented on a diff in the pull request:
https://github.com/apache/madlib/pull/339#discussion_r237326034
--- Diff: src/ports/postgres/modules/utilities/utilities.py_in ---
@@ -407,6 +407,15 @@ def set_client_min_messages(new_level):
return old_msg_level
# -------------------------------------------------------------------------
+def is_less_than_pg11(schema_madlib, **kwargs):
--- End diff --
I noticed that we have another function called `is_less_than_pg90` which
has the same code as this function. We probably don't need the pg90 function
anymore but it would be nice if the `is_less_than_pg11` function can be
generalized to take in a major postgres version and then compare the current
version with the passed major version.
---
