trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From traflm <...@git.apache.org>
Subject [GitHub] incubator-trafodion pull request #837: [TRAFODION-2355] document about IP ad...
Date Tue, 29 Nov 2016 13:22:47 GMT
Github user traflm commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/837#discussion_r90006694
  
    --- 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 --
    
    Thanks Dave for careful and patient review, this is really helpful. I will update the
words.
    For the is_ipv4(max(b)) assertion failure, I will do some debug.


---
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.
---

Mime
View raw message