Github user pmccaffrey6 commented on the issue:
https://github.com/apache/zeppelin/pull/1776
Hey @bzz,
Great catch! Sorry about that. I removed the unnecessary import. The
error seems unrelated. It comes from:
```
1470 21:03:56,259 ERROR org.apache.zeppelin.AbstractZeppelinIT:136 -
Exception in ZeppelinIT while testSparkInterpreterDependencyLoading
1471 org.openqa.selenium.TimeoutException: Timed out after 60 seconds
waiting for org.apache.zeppelin.AbstractZeppelinIT$1@7918cba6
... Stack Trace ...
1507 Caused by: org.openqa.selenium.NoSuchElementException: Unable to
locate element:
{"method":"xpath","selector":"(//div[@ng-controller=\"ParagraphCtrl\"])[1]//div[contains(@class,
'control')]//span[1][contains(.,'FINISHED')]"}
```
As for the other excellent points that @DrIgor noted (including the great
point about using logger), those issues are present both in `getMetaData()` as
well as `executeSql()` methods. Would you prefer that I make those changes to
the `getMetaData()` method in this PR or refactor both `getMetaData()` and
`executeSql()` in a separate PR?
As always, thanks for your review!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
