Skip to content
This repository has been archived by the owner on Aug 11, 2020. It is now read-only.

Support create db #80

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

jmsanchezff
Copy link
Contributor

@jmsanchezff jmsanchezff commented Jun 10, 2020

This branch adds the following functionality:

  • creation of database managed by InfluxDB instances if desired by the developer.
  • increased exception granularity allowing detect http requests issues when (on writing or querying)
  • added capability to write vector of points as a single batch
  • added precision management for floating point numbers

This branch fixes:

  • fixed issue #81
  • fixed bug when trying to write points with tag whose value is an empty string

Unit tests were included on each functionality added.

@jmsanchezff
Copy link
Contributor Author

Hi Adam,
I am not sure why testQuery is failing on Mac platform. It seems that I did not relevant changes on query class functions.

Have you some clue about that test failing?

Thanks in advance

@jmsanchezff
Copy link
Contributor Author

Hi Adam,
I am not sure why testQuery is failing on Mac platform. It seems that I did not relevant changes on query class functions.

Have you some clue about that test failing?

Thanks in advance

It seems that queryPerformance was failing on MacOs testing environment. As it consists on a performance test (not functional test) I am not really sure about that test is appropriated. My doubt is based on the fact that it is really dependent on testing environment that seems to be not properly defined for both MacOs and Linux testing environments.

What do you think? Should I recover that test for pull request to be merged to master branch?

Thank you for your comments :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant