ChanHo Lee
ChanHo Lee
I looked into the failing Selenium test to find the cause. My working hypothesis is that `maven-shade-plugin` is rewriting a string-literal, the default value of `ZEPPELIN_CONFIG_STORAGE_CLASS` inside `zeppelin-interpreter`'s `ZeppelinConfiguration`. I...
There are still some references to `zeppelin-zengine` left in the shell scripts under `/bin`. (e.g., https://github.com/jongyoul/zeppelin/blob/5cc2196bf7afc172cf6f9f7d7cc5f64f9f70b31b/bin/interpreter.sh#L115)
How about switching to the unshaded `ZeppelinConfiguration` as in [this commit](https://github.com/tbonelee/zeppelin/commit/5cf4d99be919127ca11336ecdb9b8ef45550e089)? (`2.` from my previous comment.) Maybe `rawString` fixes some cases, but by skipping required shaded classes it creates new...
On the official site, the only reference I could find is the Javadoc link for the type used by the `` property (https://maven.apache.org/components/plugins/maven-shade-plugin/apidocs/org/apache/maven/plugins/shade/relocation/SimpleRelocator.html), which doesn’t provide much detail. I skimmed...
Quick question with limited context. Why is `ZeppelinConfiguration` in the `zeppelin-interpreter` module? It looks like a general configuration class, so I wondered why it lives under interpreter. Because it sits...
@dependabot rebase
Thank you for the proposal and the PR! First of all, I agree with the idea of migrating the framework from Angular to React. This change could help lower the...
Additional requests: if you have some time, could you extract the commits which fix the E2E tests into an separate PR? We could merge that first to fix brokent tests...
I've just merged the PRs that fixed the CI failures into master. Your PR doesn't have conflicts with master, but to ensure the CI runs cleanly, could you please rebase...
Could you also take a look at this? It seems that fetching configurations in `ConfigurationService.initialize()` when the user is not logged in might be causing some issues.