-
Notifications
You must be signed in to change notification settings - Fork 961
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
sample_glue_loader returns 'Something went wrong...' on the front-end #881
Comments
do you verify the table is ingested into neo4j correctly with neo4j UI? |
I can see the tables on neo4j, when I switch to the table view, I see the similar content for all tables retrieved: |
I'm experiencing the same thing as well. Any help here |
OElesin, check the Athena one if you need to get crawl the s3 data. If you use Python, not Airflow, I slightly modified the original code at https://github.com/eponkratova/articles/blob/master/data_discovery/code/athena_data_loader.py |
@eponkratova, I tried this and got the same error. This is quite surprising . I executed the API request from the browser console and got the error below: {
"msg": "Encountered error: Metadata request failed",
"status_code": 500,
"tableData": {}
} |
From the error logs self.client = get_proxy_client()
--
File "/usr/local/lib/python3.7/site-packages/amundsen_metadata-3.0.1-py3.7.egg/metadata_service/proxy/__init__.py", line 47, in get_proxy_client
client_kwargs=client_kwargs)
TypeError: __init__() got an unexpected keyword argument 'client_kwargs' |
@feng-tao and @eponkratova, the issue is likely due to recent changes in the metadata library. I reverted the image being used to Looks like this is from: https://github.com/amundsen-io/amundsenmetadatalibrary/blob/master/metadata_service/proxy/__init__.py#L47 |
@OElesin , so, did you make the Glue connector working as well? |
@eponkratova, both are working. |
Aha, thank you, @OElesin |
cc @mgorsk1 |
I have installed new code and I am also facing the same issue and it complains about
|
@mgorsk1 could you take a look? |
sure, looking into it |
apologies, that's my blunder. I've submitted fix amundsen-io/amundsenmetadatalibrary#258 any chance you can verify if that helped @OElesin @eponkratova |
no worry, @mgorsk1 's change (amundsen-io/amundsenmetadatalibrary#258) is just merged. @NiravLangaliya could you pull from master and see if the issue is gone? |
Feel free to reopen if the issue persists after pulling from master. |
…undsen-io#881) * added PR creation to release action Signed-off-by: Allison Suarez Miranda <[email protected]> * set new branch rather than master Signed-off-by: Allison Suarez Miranda <[email protected]> * changed cron to 12:15pm Signed-off-by: Allison Suarez Miranda <[email protected]> * chnaged release aciton branch and added manual run trigger Signed-off-by: Allison Suarez Miranda <[email protected]> * eventually I'll get this right... Signed-off-by: Allison Suarez Miranda <[email protected]> * another one Signed-off-by: Allison Suarez Miranda <[email protected]> * one more branch change Signed-off-by: Allison Suarez Miranda <[email protected]> * trying to get the branch there so it can be checked out Signed-off-by: Allison Suarez Miranda <[email protected]> * added pypy credentials Signed-off-by: Allison Suarez Miranda <[email protected]> * chaged to uses/with Signed-off-by: Allison Suarez Miranda <[email protected]> * bumped version on setup.py file Signed-off-by: Allison Suarez Miranda <[email protected]> * added pypi user and pass Signed-off-by: Allison Suarez Miranda <[email protected]>
* added PR creation to release action Signed-off-by: Allison Suarez Miranda <[email protected]> * set new branch rather than master Signed-off-by: Allison Suarez Miranda <[email protected]> * changed cron to 12:15pm Signed-off-by: Allison Suarez Miranda <[email protected]> * chnaged release aciton branch and added manual run trigger Signed-off-by: Allison Suarez Miranda <[email protected]> * eventually I'll get this right... Signed-off-by: Allison Suarez Miranda <[email protected]> * another one Signed-off-by: Allison Suarez Miranda <[email protected]> * one more branch change Signed-off-by: Allison Suarez Miranda <[email protected]> * trying to get the branch there so it can be checked out Signed-off-by: Allison Suarez Miranda <[email protected]> * added pypy credentials Signed-off-by: Allison Suarez Miranda <[email protected]> * chaged to uses/with Signed-off-by: Allison Suarez Miranda <[email protected]> * bumped version on setup.py file Signed-off-by: Allison Suarez Miranda <[email protected]> * added pypi user and pass Signed-off-by: Allison Suarez Miranda <[email protected]>
@feng-tao - I'm getting the same issue now with loading Glue data. I'm on the latest branch from master, my data is in neo4j. When I click on a table in the WebUI I get 404 not found. Any guidance would be appreciated! |
…undsen-io#881) * added PR creation to release action Signed-off-by: Allison Suarez Miranda <[email protected]> * set new branch rather than master Signed-off-by: Allison Suarez Miranda <[email protected]> * changed cron to 12:15pm Signed-off-by: Allison Suarez Miranda <[email protected]> * chnaged release aciton branch and added manual run trigger Signed-off-by: Allison Suarez Miranda <[email protected]> * eventually I'll get this right... Signed-off-by: Allison Suarez Miranda <[email protected]> * another one Signed-off-by: Allison Suarez Miranda <[email protected]> * one more branch change Signed-off-by: Allison Suarez Miranda <[email protected]> * trying to get the branch there so it can be checked out Signed-off-by: Allison Suarez Miranda <[email protected]> * added pypy credentials Signed-off-by: Allison Suarez Miranda <[email protected]> * chaged to uses/with Signed-off-by: Allison Suarez Miranda <[email protected]> * bumped version on setup.py file Signed-off-by: Allison Suarez Miranda <[email protected]> * added pypi user and pass Signed-off-by: Allison Suarez Miranda <[email protected]>
…undsen-io#881) * added PR creation to release action Signed-off-by: Allison Suarez Miranda <[email protected]> * set new branch rather than master Signed-off-by: Allison Suarez Miranda <[email protected]> * changed cron to 12:15pm Signed-off-by: Allison Suarez Miranda <[email protected]> * chnaged release aciton branch and added manual run trigger Signed-off-by: Allison Suarez Miranda <[email protected]> * eventually I'll get this right... Signed-off-by: Allison Suarez Miranda <[email protected]> * another one Signed-off-by: Allison Suarez Miranda <[email protected]> * one more branch change Signed-off-by: Allison Suarez Miranda <[email protected]> * trying to get the branch there so it can be checked out Signed-off-by: Allison Suarez Miranda <[email protected]> * added pypy credentials Signed-off-by: Allison Suarez Miranda <[email protected]> * chaged to uses/with Signed-off-by: Allison Suarez Miranda <[email protected]> * bumped version on setup.py file Signed-off-by: Allison Suarez Miranda <[email protected]> * added pypi user and pass Signed-off-by: Allison Suarez Miranda <[email protected]>
Expected Behavior
Glue tables metadata is displayed on the front-end
Current Behavior
On the front-end, I see 'Something went wrong...'
Possible Solution
Steps to Reproduce
I am running Amundsen on my local machine:
http://localhost:5000/table_detail/AWSSessionToken=%22/glue/iris/iriscsv.
Screenshots (if appropriate)
Context
I am doing a POC.
Your Environment
The text was updated successfully, but these errors were encountered: