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

"An error has occurred: null" when importing module from shell #918

Open
ingomueller-net opened this issue Feb 11, 2021 · 2 comments
Open

Comments

@ingomueller-net
Copy link
Contributor

I can't seem to import modules with starting Rumble with --shell yes. Running exactly the same query from a file using --query-path works. I have tried a couple of different modules and none of them seems to work.

bash$ cat /path/test.jq
import module namespace hep = "/path/module.jq"; 42
bash$ docker run --rm -it -v $PWD:$PWD rumbledb/rumble:v1.10.0-spark3 --query-path /path/test.jq
42
bash$ docker run --rm -it -v $PWD:$PWD rumbledb/rumble:v1.10.0-spark3 --shell yes
...
rumble$ import module namespace hep = "/path/module.jq"; 42
>>>
>>>
[ERROR] An error has occurred: null
We should investigate this 🙈. Please contact us or file an issue on GitHub with your query.
Link: https://github.com/RumbleDB/rumble/issues
@ghislainfourny
Copy link
Member

ghislainfourny commented Feb 24, 2021

Hi Ingo, I could not reproduce. On my side it either works out of the box, or appropriately returns a not-found error code.

Does it work without docker?

Also, could you try to run this again with --show-error-info yes?

It might be docker works with internal paths, then it cannot see what is on your file system, you need to forward the paths in some docker-specific way. Regardless, I need to fix the null pointer exception.

@ingomueller-net
Copy link
Contributor Author

The same happens if I use v1.10.0 compiled by myself and run against Spark 2.4 outside Docker.

(Note that the command above mounts the current folder into the docker container at the same path, so all absolute paths below the current folder should work in the container as well.)

@ingomueller-net ingomueller-net removed their assignment Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants