looker-open-source / henry

A command line tool for Looker instance cleanup
MIT License
79 stars 27 forks source link

`henry analyze explores` run problem #5

Closed sfrattura closed 4 years ago

sfrattura commented 4 years ago

@deangelo-llooker

i run this as follows on a small model, and it works fine: henry analyze explores --model promo_calculator --timeframe 2 --host prod_looker

i then try it on a slightly larger model henry analyze explores --model services_data --timeframe 2 --host prod_looker

... and after a while, I get this error (redacting our server name): |Error: 502 Server Error: Bad Gateway for url: https://:19999/api/3.0/queries/run/json Traceback (most recent call last): File "/opt/python_virtualenvs/henry/bin/henry", line 10, in sys.exit(main()) File "/opt/python_virtualenvs/henry/lib/python3.6/site-packages/henry/cli.py", line 291, in main result = analyze.analyze(**args) File "/opt/python_virtualenvs/henry/lib/python3.6/site-packages/henry/commands/analyze.py", line 52, in analyze min_queries=kwargs['min_queries']) File "/opt/python_virtualenvs/henry/lib/python3.6/site-packages/henry/commands/analyze.py", line 137, in _analyze_explores explore=e['name']) File "/opt/python_virtualenvs/henry/lib/python3.6/site-packages/henry/modules/fetcher.py", line 239, in get_used_explores for r in response: TypeError: 'NoneType' object is not iterable

We will dig on this a little on our end -- but any suggestions you have would be great

josephaxisa commented 4 years ago

@sfrattura, did you get this solved? It seems that it's trying to iterate over an empty list.

sfrattura commented 4 years ago

No. I have not resolved this. But to be fair, I have not run it again since. I know it is a problem..and I can not solve it (I'm not an advanced Python engineer)

:-(

On Tue, Nov 12, 2019 at 7:26 AM Joseph Axisa notifications@github.com wrote:

@sfrattura https://github.com/sfrattura, did you get this solved? It seems that it's trying to iterate over an empty list.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/looker-open-source/henry/issues/5?email_source=notifications&email_token=ANGYB3TORVPI2245DMTSG4TQTKOHHA5CNFSM4JFFN3AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED2CR5Y#issuecomment-552872183, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANGYB3ULFK3YSIAWJYAFOC3QTKOHHANCNFSM4JFFN3AA .

josephaxisa commented 4 years ago

@sfrattura I just pushed an update, please try updating and see if the problem persists.

sfrattura commented 4 years ago

will do. thanks! I'll keep you posted

On Tue, Feb 18, 2020 at 3:37 PM Joseph Axisa notifications@github.com wrote:

@sfrattura https://github.com/sfrattura I just pushed an update, please try updating and see if the problem persists.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/looker-open-source/henry/issues/5?email_source=notifications&email_token=ANGYB3VBAKS3K54FRTDEZW3RDRBIZA5CNFSM4JFFN3AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMETNYQ#issuecomment-587806434, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANGYB3RILYYH4LALZ4T6WJLRDRBIZANCNFSM4JFFN3AA .

josephaxisa commented 4 years ago

Going to assume this has been resolved with the latest release. Please let me know if you experience this again.

alecburnett commented 2 years ago

This hasn't been resolved. Using python 3.7 and henry-0.2.3 and looker-sdk-0.1.3b7 on a mac.