Skip to content

[IOTDB-31] failed to set float precision when creating timeseries#80

Closed
MyXOF wants to merge 1 commit intomasterfrom
fix_float_precision_bug
Closed

[IOTDB-31] failed to set float precision when creating timeseries#80
MyXOF wants to merge 1 commit intomasterfrom
fix_float_precision_bug

Conversation

@MyXOF
Copy link
Member

@MyXOF MyXOF commented Feb 28, 2019

No description provided.

@jixuan1989
Copy link
Member

If you want to avoid for, modify LogicalGenerator.analyzeMetadataCreate() to transfer all keywords of props as lowercase.

But I suggest that you modify this on my PR... because there are heavy conflicts...

LOGGER.warn("cannot set max point number to negative value, replaced with default value:{}",
maxPointNumber);
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that the else block is just enough and the if block is redundant.
You may set maxPointNumber = TSFileConfig.floatPrecision first and then go through the loop for(Map.Entry<String, String> entry : props.entrySet()) .

@MyXOF MyXOF closed this Mar 1, 2019
@MyXOF MyXOF deleted the fix_float_precision_bug branch March 1, 2019 08:06
mychaow pushed a commit that referenced this pull request Dec 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants