[GitHub] incubator-trafodion pull request #837: [TRAFODION-2355] document about IP ad...

[DaveBirdsall <gi...@git.apache.org>]

Github user DaveBirdsall commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/837#discussion_r89910126
  
    --- Diff: docs/sql_reference/src/asciidoc/_chapters/sql_functions_and_expressions.adoc ---
    @@ -3658,12 +3658,10 @@ IS_IPV4( expression )
     * `_expression_`
     +
     specifies an expression that determines the values to include in the
    -computation of the minimum. The _expression_ cannot contain an aggregate
    -function or a subquery. The DISTINCT clause specifies that the MIN
    -function operates on distinct values from the one-column table derived
    -from the evaluation of _expression_. All nulls are eliminated before
    -the function is applied to the set of values. If the result table is
    -empty, IS_IPV4 returns NULL.
    +validation of the IP address. The _expression_ cannot contain an aggregate
    +function or a subquery. All nulls are eliminated before the function is
    +applied to the set of values. If the result table is empty, IS_IPV4 
    --- End diff --
    
    The last two sentences appear to be discussing sets or tables of values, but IS_IPV4 just takes a single value. Consider replacing "All nulls... returns NULL" with "If the input value is NULL, IS_IPV4 returns NULL." Incidentally, just for fun I tried doing select is_ipv4(max(b)) from temp1, where temp1 is a table containing a char column b, and I got an assertion failure.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---