Skip to content

Conversation

@mirza99
Copy link
Contributor

@mirza99 mirza99 commented Sep 11, 2018

Enable adding fields by passing a POJO to the Builder.

How to use it

// Order is a POJO having annotations @Measurement and @Column
Order order = new Order();
// ... setting data

Point p = Point.measurementByPOJO(order.getClass()).addFieldsFromPOJO(order).build();

@codecov-io
Copy link

codecov-io commented Sep 11, 2018

Codecov Report

Merging #519 into master will increase coverage by 0.03%.
The diff coverage is 90.9%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #519      +/-   ##
============================================
+ Coverage     88.05%   88.09%   +0.03%     
- Complexity      697      703       +6     
============================================
  Files            68       69       +1     
  Lines          2378     2411      +33     
  Branches        240      245       +5     
============================================
+ Hits           2094     2124      +30     
- Misses          198      201       +3     
  Partials         86       86
Impacted Files Coverage Δ Complexity Δ
src/main/java/org/influxdb/BuilderException.java 100% <100%> (ø) 1 <1> (?)
src/main/java/org/influxdb/dto/Point.java 85.34% <90.32%> (+0.96%) 40 <5> (+5) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3e4e335...be2c3b7. Read the comment docs.

@fmachado fmachado mentioned this pull request Oct 1, 2018
@fmachado
Copy link
Contributor

@mirza99 I would like to merge this PR but some checks are failing. Would be possible to fix them? Thanks again for your contribution!

@mirza99
Copy link
Contributor Author

mirza99 commented Oct 29, 2018

@fmachado You can merge the PR now. Sorry for the delay.

@fmachado fmachado self-assigned this Oct 29, 2018
@fmachado fmachado merged commit 1dafcc1 into influxdata:master Feb 21, 2019
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