Hi,

Thank you for taking on this issue :)

Have you looked at client_test_with_server.py? I would like some DaframeClient tests there.

You are using the u'string' syntax, which does not work in python 3.2.

Other than that, this looks great!

//cc @timtroendle

@ReAzem In case you haven't reviewed it, a few basic tests were added in client_test_with_server.py in this pull request as suggested. The 3.2 syntax problem is also fixed. I've also changed the key data structure for data dictionary's keys, for both querying and submitting, where strings can be used as keys if no tags are used. Otherwise, the keys can be a tuple of (measurement, sorted tuple of tag key-value pairs).

@tzonghao Hello,

Thank you for the great work. Why are you not using the ResultSet Object? You should not have to use the raw property.

@ReAzem You're right. Using ResultSet is cleaner. It's converted in the last commit.

@tzonghao Can you change the write_points function to accept three arguments instead?

• measurment name
• tags (optional)
• dataframe

I think it would be more intuitive than a tuple of key-value tuples.

That would be my last request before we merge :)

@ReAzem Sure, I can make that change. It also implicates that write_points will accept one DataFrame at a time, instead of multiple DataFrames.

@tzonghao I think it would be worth it, do you agree?

I agree. It's a good decision.

@ReAzem It looks like an InfluxDB error in the last Travis run that failed the py34 test. Can you trigger a rerun?

Great work! I will be merging this for now. I'm sure we can keep on improving it after feedback from other users.