[GitHub] [druid] gianm commented on a change in pull request #10635: Add SQL functions to format numbers into human readable format

2021-01-09 Thread GitBox


gianm commented on a change in pull request #10635:
URL: https://github.com/apache/druid/pull/10635#discussion_r554228178



##
File path: docs/querying/sql.md
##
@@ -563,6 +563,9 @@ The [DataSketches 
extension](../development/extensions-core/datasketches-extensi
 |`COALESCE(value1, value2, ...)`|Returns the first value that is neither NULL 
nor empty string.|
 |`NVL(expr,expr-for-null)`|Returns 'expr-for-null' if 'expr' is null (or empty 
string for string type).|
 |`BLOOM_FILTER_TEST(, )`|Returns true if the value is 
contained in a Base64-serialized bloom filter. See the [Bloom filter 
extension](../development/extensions-core/bloom-filter.html) documentation for 
additional details.|
+|`BINARY_BYTE_FORMAT(value, [precision])`|Returns the value in human-readable 
[IEC](https://en.wikipedia.org/wiki/Binary_prefix) format. Supported unit 
suffix: `B`, `KiB`, `MiB`, `GiB`, `TiB`, `PiB`, `EiB`. `precision` must be in 
the range of [0,3] (default: 2).|
+|`DECIMAL_BYTE_FORMAT(value, [precision])`|Returns the value in human-readable 
[SI](https://en.wikipedia.org/wiki/Binary_prefix) format. Supported unit 
suffix: `B`, `KB`, `MB`, `GB`, `TB`, `PB`, `EB`. `precision` must be in the 
range of [0,3] (default: 2).|
+|`DECIMAL_FORMAT(value, [precision])`|Returns the value in human-readable SI 
format. Supported unit suffix: `K`, `M`, `G`, `T`, `P`, `E`. `precision` must 
be in the range of [0,3] (default: 2).|

Review comment:
   I think they make sense in the numeric section, because they accept 
numbers. It's OK that they don't return numbers. It's analogous to 
`TIME_FORMAT`, which still belongs in the time section.





This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org



[GitHub] [druid] gianm commented on a change in pull request #10635: Add SQL functions to format numbers into human readable format

2021-01-08 Thread GitBox


gianm commented on a change in pull request #10635:
URL: https://github.com/apache/druid/pull/10635#discussion_r554228178



##
File path: docs/querying/sql.md
##
@@ -563,6 +563,9 @@ The [DataSketches 
extension](../development/extensions-core/datasketches-extensi
 |`COALESCE(value1, value2, ...)`|Returns the first value that is neither NULL 
nor empty string.|
 |`NVL(expr,expr-for-null)`|Returns 'expr-for-null' if 'expr' is null (or empty 
string for string type).|
 |`BLOOM_FILTER_TEST(, )`|Returns true if the value is 
contained in a Base64-serialized bloom filter. See the [Bloom filter 
extension](../development/extensions-core/bloom-filter.html) documentation for 
additional details.|
+|`BINARY_BYTE_FORMAT(value, [precision])`|Returns the value in human-readable 
[IEC](https://en.wikipedia.org/wiki/Binary_prefix) format. Supported unit 
suffix: `B`, `KiB`, `MiB`, `GiB`, `TiB`, `PiB`, `EiB`. `precision` must be in 
the range of [0,3] (default: 2).|
+|`DECIMAL_BYTE_FORMAT(value, [precision])`|Returns the value in human-readable 
[SI](https://en.wikipedia.org/wiki/Binary_prefix) format. Supported unit 
suffix: `B`, `KB`, `MB`, `GB`, `TB`, `PB`, `EB`. `precision` must be in the 
range of [0,3] (default: 2).|
+|`DECIMAL_FORMAT(value, [precision])`|Returns the value in human-readable SI 
format. Supported unit suffix: `K`, `M`, `G`, `T`, `P`, `E`. `precision` must 
be in the range of [0,3] (default: 2).|

Review comment:
   I think they make sense in the numeric section, because they accept 
numbers. It's OK that they don't return numbers. It's analogous to 
`TIME_FORMAT`, which still belongs in the time section.





This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org