Skip to content
This repository has been archived by the owner on Oct 29, 2024. It is now read-only.

Add functions to SHOW TAG KEYS and SHOW FIELD KEYS #779

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

Conversation

tuykroa
Copy link

@tuykroa tuykroa commented Feb 1, 2020

Then supported these 2 new functions : SHOW TAG KEYS & SHOW FIELD KEYS
like InfluxDB show field keys in query-language
https://docs.influxdata.com/influxdb/v1.7/query_language/schema_exploration/#show-field-keys

[x]Controlled

If you accept my contribution to this feature

Copy link
Author

@tuykroa tuykroa left a comment

Choose a reason for hiding this comment

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

Due error, submit changes without spaces in comments

@tuykroa
Copy link
Author

tuykroa commented Feb 1, 2020

Then supported these 2 new functions : SHOW TAG KEYS & SHOW FIELD KEYS
like InfluxDB show field keys in query-language
https://docs.influxdata.com/influxdb/v1.7/query_language/schema_exploration/#show-field-keys

[x]Controlled
[x]Verified
If you accept my contribution to this feature

@sebito91 sebito91 self-assigned this Apr 8, 2020
@sebito91
Copy link
Contributor

Thanks for the commit @tuykroa, really appreciate your help! If you're still interested in merging this PR can I please ask you to do the following:

  • flush out the show tag keys command similar to what the documentation calls for, including capability to drill down on specific db or measurement
  • flush out the show field keys command similar to the above
  • add tests for show tag keys
  • add tests for show field keys

You can use

def get_list_series(self, database=None, measurement=None, tags=None):
as an example.

Thanks!

@sebito91
Copy link
Contributor

sebito91 commented Jun 4, 2020

@tuykroa just following up to see if you were able to make some progress here?

@one4many
Copy link

I'm depending on this ... when does it get reviewed?

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

Successfully merging this pull request may close these issues.

3 participants