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

ElasticSearch Driver: The first argument must be one of type string, Buffer, ArrayBuffer, Array, or Array-like Object. Received type undefined #614

Open
chadfaurie opened this issue Apr 24, 2020 · 5 comments
Labels
community supported driver:elasticsearch Issues relating to the ElasticSearch driver enhancement New feature proposal help wanted Community contributions are welcome.

Comments

@chadfaurie
Copy link
Contributor

Describe the bug
When Running the elasticsearch driver in the latest version I get the following:
The first argument must be one of type string, Buffer, ArrayBuffer, Array, or Array-like Object. Received type undefined

To Reproduce
Steps to reproduce the behavior:
Use ElasticSearch Cloud driver with demo data and CubeJS playground

Expected behavior
No error on requesting data from elastic

Screenshots
If applicable, add screenshots to help explain your problem.

Version:
0.19.13

Additional context

@chadfaurie
Copy link
Contributor Author

This issue is caused by passing empty strings in Client initializer for ElasticSearch

@chadfaurie
Copy link
Contributor Author

Should maybe have a graceful way of notifying the user?

@chadfaurie chadfaurie reopened this Apr 24, 2020
@paveltiunov
Copy link
Member

@chad-codeworkshop Hey Chad! Do you mean we need add sanity check to see if arguments passed? I believe it'd be helpful.

@paveltiunov paveltiunov added the enhancement New feature proposal label Apr 24, 2020
@chadfaurie
Copy link
Contributor Author

@chad-codeworkshop Hey Chad! Do you mean we need add sanity check to see if arguments passed? I believe it'd be helpful.

Yeah, i spent about an hour tracking down the issue and it was something as simple as environment variables not being set, so the elastic Client failed

I will start having a look at how to implement this in the coming week

@hassankhan hassankhan added the help wanted Community contributions are welcome. label Dec 5, 2020
@github-actions
Copy link

github-actions bot commented Dec 5, 2020

If you are interested in working on this issue, please leave a comment below and we will be happy to assign the issue to you.
If this is the first time you are contributing a Pull Request to Cube.js, please check our contribution guidelines.
You can also post any questions while contributing in the #contributors channel in the Cube.js Slack.

@igorlukanin igorlukanin added the driver:elasticsearch Issues relating to the ElasticSearch driver label Sep 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community supported driver:elasticsearch Issues relating to the ElasticSearch driver enhancement New feature proposal help wanted Community contributions are welcome.
Projects
None yet
Development

No branches or pull requests

4 participants