[HOTFIX] ZEPPELIN-931: fix interpreter listing bug#945
[HOTFIX] ZEPPELIN-931: fix interpreter listing bug#945AhyoungRyu wants to merge 5 commits intoapache:masterfrom
Conversation
|
@jongyoul I saw you are working on applying new mechanism to all of the existing interpreters in ZEPPELIN-804 as subtasks. I think this bug was caused since |
|
@AhyoungRyu I've tested and reproduced what you mentioned. Your change looks simple but it's not perfect because you cannot see spark interpreter in your interpreter list because interpreter-setting.json read |
|
I don't know whether front-end uses RegisteredInterprerter. We need to find a way to indicate or mark which class is used in front and backend. |
|
@jongyoul I see. That's why I can't see |
|
@AhyoungRyu Could you please add "[HOTFIX]" in a title? |
|
@jongyoul Yeah I just added :) |
Fixed fieldName in interpreter-setting.json
|
@jongyoul I applied your patch and it works :) |
|
@AhyoungRyu I've missed something. I submitted a PR, again. |
Fixed documentation
|
@jongyoul That was also applied. |
|
@AhyoungRyu Thanks. |
|
@jongyoul Let's wait the travis then :) |
|
@Leemoonsoo CI has been passed. Could you check this one ? :) |
|
LGTM. I'll merge it asap. |
|
@jongyoul Yeah sounds good. |
What is this PR for?
Currently available interpreter list is not shown in
Creating New Interpretersection. It seems this bug was generated after #835 was merged. So I temporally deactivated 3 SerializedName code lines.What type of PR is it?
Bug Fix
Todos
What is the Jira issue?
ZEPPELIN-931
How should this be tested?
Screenshots (if appropriate)
Questions: