-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Ambari 22897 Tez view button doesnt works in ambari hiveviews 1.5 #408
Conversation
Refer to this link for build results (access rights to CI server needed): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add licence to "contrib/views/hive-next/src/main/resources/ui/hive-web/app/components/query-settings.js" and do not use -Drat.skip when using mvn clean install
Refer to this link for build results (access rights to CI server needed): |
…he Unused code, layout settings tab correctly(asnaik)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LGTM. |
Refer to this link for build results (access rights to CI server needed): |
What changes were proposed in this pull request?
Ambari 22897 - Tez view button doesnt works in ambari hive views 1.5
Visual-explain, visualization , tez , and settings was 'views' is not made route due to those buttons where not working as it was hitting : emberjs/ember.js#10780
How was this patch tested?
Tested the UI changes
All works fine
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 57.113 s
[INFO] Finished at: 2018-02-20T16:31:12+05:30
[INFO] Final Memory: 62M/521M
UT cases corrected and folder made correct
[INFO]
[INFO] Tests run: 94, Failures: 0, Errors: 0, Skipped: 0
No change to Apperance even though there are changes in the implementation logic.