apache/pinot

Add read APIs for `BOOLEAN` and `TIMESTAMP`

Open

#9.418 geöffnet am 16. Sept. 2022

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (2 zugewiesene Personen)Java (1.234 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (4.937 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 7h) (186 gemergte PRs in 30 T)

Beschreibung

Currently BOOLEAN is supported using the INT data type, and TIMESTAMP is supported using the LONG data type. Even though the storage format are the same, the behavior should be different. E.g. reading BOOLEAN as STRING should give "true" or "false" instead of 1 or 0; reading DOUBLE as BOOLEAN should give doubleVal != 0 instead of (int) doubleVal. In order to achieve the expected behavior, we need to add read APIs for these 2 data types. It should be similar to how the BIG_DECIMAL is handled (storage format is BYTES). We want to add type support in:

  • Dictionary
  • ForwardIndexReader
  • BlockValSet
  • TransformFunction

Contributor Guide