This is an automated email from the ASF dual-hosted git repository. snlee pushed a commit to branch revert-12032-santizeNanNegativeZeroBehaviour in repository https://gitbox.apache.org/repos/asf/pinot.git
commit 15e41e1089c37d4aa352220ed11f12a1ba9d78f9 Author: Seunghyun Lee <[email protected]> AuthorDate: Mon Nov 27 11:39:57 2023 -0800 Revert "fixed note in SpecialValueTransformer." This reverts commit f79b618141e07c140663791959d96956ad91d811. --- .../segment/local/recordtransformer/SpecialValueTransformer.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/SpecialValueTransformer.java b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/SpecialValueTransformer.java index 29596e0935..3383803194 100644 --- a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/SpecialValueTransformer.java +++ b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/SpecialValueTransformer.java @@ -35,9 +35,8 @@ import org.slf4j.LoggerFactory; * <li>Negative zero (-0.0) should be converted to 0.0</li> * <li>NaN should be converted to default null</li> * </ul> - * <p>NOTE: should put this after the {@link DataTypeTransformer} so that we already have the values complying - * with the schema before handling special values and before {@link NullValueTransformer} so that it transforms - * all the null values properly. + * <p>NOTE: should put this after the {@link DataTypeTransformer} so that all values follow the data types in + * {@link FieldSpec}. */ public class SpecialValueTransformer implements RecordTransformer { --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
