Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds ability to store attributes in their own columns #33

Closed
wants to merge 6 commits into from

Conversation

sklarsa
Copy link

@sklarsa sklarsa commented Mar 26, 2024

First of all, thanks for this project! As someone who works for QuestDB and also just bought a house, this has been a lifesaver for me in starting my homeassistant journey!

Since our ILP protocol creates columns when it encounters unknowns, I figured that it might be useful to split out the attributes column into multiple columns for easier querying purposes. I left the default behavior the same, just added the split_attributes config option.

This is running successfully on my homeassistant, but I'm happy to keep this running for a little bit to ensure its stability.


EDIT: Ah, and I already hit an issue with a list value :-( Maybe unstructured storage is the way to go here for now

@sklarsa sklarsa closed this Mar 27, 2024
@CM000n
Copy link
Owner

CM000n commented Mar 28, 2024

Hi @sklarsa, never mind 😉 Thank you very much for your contribution and your ideas. Feel free to share if you have any more of them!

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