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

updated readme #37

Merged
merged 1 commit into from
Jan 4, 2021
Merged

updated readme #37

merged 1 commit into from
Jan 4, 2021

Conversation

anirudha
Copy link
Contributor

updated readme with naming changes and links to documentation for Amazon Elasticsearch service and Open Distro

updated readme with naming changes and links to documentation for Amazon Elasticsearch service and Open Distro
@anirudha
Copy link
Contributor Author

anirudha commented Dec 1, 2020

@robdiciuccio @syamat @mistercrunch @dioptre @srinify
please help here on this PR

let us know if you need us to help in any way on this feature support. My team maintains and own the opendistro SQL plugin

@dpgaspar
Copy link
Member

dpgaspar commented Jan 4, 2021

@anirudha,

Feel free to reach out, would really love to fully support opendistro but the results from the SQL endpoint are totally different from the Elasticsearch endpoint (unless this has changed recently). It's more complex to transform pure ES results to a row structure, and that's whats missing

@dpgaspar dpgaspar merged commit 40be411 into preset-io:master Jan 4, 2021
@dpgaspar dpgaspar mentioned this pull request Jan 5, 2021
@dioptre
Copy link
Contributor

dioptre commented Jan 5, 2021

@dpgaspar would it be best to fork the project? I haven't been able to use dbapi successfully at all so far.

Would be nice to understand from your perspective what's needed.

@dpgaspar
Copy link
Member

dpgaspar commented Jan 6, 2021

@dioptre do you have any open issue?

@dioptre
Copy link
Contributor

dioptre commented Jan 6, 2021

Here's my issue that's been open for 5 months.

#31

Thanks @dpgaspar

@syamat
Copy link

syamat commented Jan 7, 2021

I would also like to contribute to have opendistro support enabled. I have filed the below issue.

#5

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.

4 participants