Skip to content

Conversation

@lebuni
Copy link
Contributor

@lebuni lebuni commented Nov 30, 2025

Proposed Changes

This PR introduces dedicated methods for working with DataFrames (write_dataframe() and query_dataframe()), providing a cleaner API with better documentation and type checking compared to the previous approach of using the internal _write_api directly.
Fixes a bug introduced in PR #158 where writing DataFrames would fail with a TypeError.

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • A test has been added if appropriate
  • Tests pass
  • Commit messages are conventional
  • Sign CLA (if not already signed)

Dataframes were handled before by the general write command, but not documented and not typechecked.
Therefore implemented separate functions for writing and querying dataframes.
@lebuni lebuni force-pushed the add-specific-pandas-methods branch from 7f20d81 to 8e4dcd8 Compare November 30, 2025 16:21
Copy link
Member

@bednar bednar left a comment

Choose a reason for hiding this comment

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

@lebuni thanks for your PR 👍

There is only one requirement about CHANGELOG.md:

PR InfluxCommunity#158 introduced a bug writing dataframes.
Filtering now the kwargs before forwarding them to lower levels
@lebuni lebuni force-pushed the add-specific-pandas-methods branch from 8e4dcd8 to bae3905 Compare December 2, 2025 11:34
@lebuni lebuni requested a review from bednar December 2, 2025 11:34
Copy link
Member

@bednar bednar left a comment

Choose a reason for hiding this comment

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

@lebuni thanks again for your PR!

LGTM 🚀

@bednar bednar changed the title Add Dedicated DataFrame Methods for Writing and Querying feat: Add Dedicated DataFrame Methods for Writing and Querying Dec 2, 2025
@lebuni
Copy link
Contributor Author

lebuni commented Dec 2, 2025

Thanks @bednar for the review!

@bednar bednar merged commit fb7b3f9 into InfluxCommunity:main Dec 2, 2025
3 of 4 checks passed
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.

2 participants