asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yingyi Bu (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: Revise builtin function documents.
Date Sat, 01 Oct 2016 04:55:01 GMT
Yingyi Bu has posted comments on this change.

Change subject: Revise builtin function documents.
......................................................................


Patch Set 4:

(20 comments)

https://asterix-gerrit.ics.uci.edu/#/c/1221/4/asterixdb/asterix-doc/src/main/markdown/builtins/11_others.md
File asterixdb/asterix-doc/src/main/markdown/builtins/11_others.md:

Line 113:         len(["ASTERIX", "Hyracks"])
> More generic strings?
Done


Line 222:     * `expr2`: an expression.
> Consistent spacing?
Done


Line 231:         deep_equal(
> That's a lot of indentation ...
Done


https://asterix-gerrit.ics.uci.edu/#/c/1221/4/asterixdb/asterix-doc/src/main/markdown/builtins/1_numeric.md
File asterixdb/asterix-doc/src/main/markdown/builtins/1_numeric.md:

Line 205:  * Computes the e<sup>numeric_value</sup>.
> s/the//
Done


Line 260:  * Computes the log<sub>e</sub>numeric_value.
> s/the//
Done


Line 284:  * Computes the log<sub>10</sub>numeric_value.
> s/the//
Done


Line 308:  * Computes the numeric_value1<sup>numeric_value2</sup>.
> s/the//
Done


https://asterix-gerrit.ics.uci.edu/#/c/1221/4/asterixdb/asterix-doc/src/main/markdown/builtins/2_string.md
File asterixdb/asterix-doc/src/main/markdown/builtins/2_string.md:

Line 608:         substring_before(" like iphone", "iphone");
> expected result?
Done


Line 634:         " is great"
> Seems wrong.
Done


Line 688:  * The expected result is:
> ??
Done


https://asterix-gerrit.ics.uci.edu/#/c/1221/4/asterixdb/asterix-doc/src/main/markdown/builtins/3_binary.md
File asterixdb/asterix-doc/src/main/markdown/builtins/3_binary.md:

Line 43:       [ hex("ABCDEF0123456789"), hex("ABCDEF0123456789"), hex("4173746572697801")
]
> This looks a little strange - seems that we should not allow printing of bi
I'm not sure what the problem is for print binaries?
They are just one adm type, like datetime?


https://asterix-gerrit.ics.uci.edu/#/c/1221/4/asterixdb/asterix-doc/src/main/markdown/builtins/7_allens.md
File asterixdb/asterix-doc/src/main/markdown/builtins/7_allens.md:

Line 24:         interval_before(interval1, interval2)
> Do we cover intervals in the data model doc?
Yes, its' here:

https://ci.apache.org/projects/asterixdb/aql/datamodel.html#PrimitiveTypesInterval


https://asterix-gerrit.ics.uci.edu/#/c/1221/4/asterixdb/asterix-doc/src/main/markdown/builtins/8_record.md
File asterixdb/asterix-doc/src/main/markdown/builtins/8_record.md:

Line 149:  * Add fields from a record given a list of field names.
> s/from/to/
Done


https://asterix-gerrit.ics.uci.edu/#/c/1221/4/asterixdb/asterix-doc/src/main/markdown/builtins/9_aggregate_aql.md
File asterixdb/asterix-doc/src/main/markdown/builtins/9_aggregate_aql.md:

Line 104:  * Gets the min value of numeric items in the given collection.
> Doesn't work for other comparable values?
Done


https://asterix-gerrit.ics.uci.edu/#/c/1221/4/asterixdb/asterix-doc/src/main/markdown/builtins/9_aggregate_sql.md
File asterixdb/asterix-doc/src/main/markdown/builtins/9_aggregate_sql.md:

Line 32:         array_count(collection)
> This should follow our latest array_ vs coll_ discussion.
Done


Line 109:  * Gets the min value of numeric items in the given collection.
> Doesn't work for other comparable values?
Done


Line 137:  * Gets the max value of numeric items in the given collection.
> Doesn't work for other comparable values?
Done


Line 245:  * Gets the min value of non-null and non-missing numeric items in the given collection.
> Doesn't work for other comparable values?
Done


Line 273:  * Gets the max value of the non-null and non-missing numeric items in the given
collection.
> Doesn't work for other comparable values?
Done


https://asterix-gerrit.ics.uci.edu/#/c/1221/4/asterixdb/asterix-doc/src/main/markdown/sqlpp/3_query.md
File asterixdb/asterix-doc/src/main/markdown/sqlpp/3_query.md:

Line 596: | ARRAY_COUNT     | counted      | counted      | 0                |
> Should follow our latest ARRAY_ vs COLL_ discussion.
Done


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1221
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4b05108b1bc741585717192b0b721ad00959bd83
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <buyingyi@gmail.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>
Gerrit-Reviewer: Yingyi Bu <buyingyi@gmail.com>
Gerrit-HasComments: Yes

Mime
View raw message