---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57136/#review167100
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 28, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57120/#review167066
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 28, 2017, 4
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57126/#review167065
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 28, 2017, 6
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57124/#review167064
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 28, 2017, 5
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57143/#review167060
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 28, 2017, 1
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57130/#review167057
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 28, 2017, 9
,
Pallav Kulshreshtha, and Rohit Choudhary.
Bugs: AMBARI-19673
https://issues.apache.org/jira/browse/AMBARI-19673
Repository: ambari
Description
---
User should be able to see code hints and syntax colorization while typing hive
queries.
Diffs
-
contrib/views/hive20/src/main
/app/services/query.js 3bf10c0
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query/results.hbs
ad44c5a
Diff: https://reviews.apache.org/r/57131/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/hive20/src/main/resources/ui/app/templates/queries/query/results.hbs
ad44c5a
Diff: https://reviews.apache.org/r/57131/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57041/#review166816
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 24, 2017, 2
/hive20/src/main/resources/ui/app/templates/queries/query/visual-explain.hbs
5b7e566
Diff: https://reviews.apache.org/r/57019/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
Diff: https://reviews.apache.org/r/57036/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
,
Pallav Kulshreshtha
/hive20/src/main/resources/ui/app/templates/queries/query/visual-explain.hbs
ef441b5
Diff: https://reviews.apache.org/r/57019/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/query/visual-explain.hbs
ef441b5
Diff: https://reviews.apache.org/r/57019/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56973/#review166490
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 23, 2017, 7
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56971/#review166489
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 23, 2017, 6
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56972/#review166488
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 23, 2017, 6
0/src/main/resources/ui/app/routes/queries/query.js
64f38ab
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
4fd3ce201
Diff: https://reviews.apache.org/r/56965/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/hive-explainer/renderer-force.js
2dfdc86
contrib/views/hive20/src/main/resources/ui/app/utils/hive-explainer/renderer.js
a332802
Diff: https://reviews.apache.org/r/56925/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
2dfdc86
contrib/views/hive20/src/main/resources/ui/app/utils/hive-explainer/renderer.js
a332802
Diff: https://reviews.apache.org/r/56925/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56882/#review166169
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 21, 2017
Diff: https://reviews.apache.org/r/56884/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/hive20/src/main/resources/ui/app/templates/queries/query.hbs
d62c781
contrib/views/hive20/src/main/resources/ui/app/utils/hive-explainer/renderer.js
3dedd8f
Diff: https://reviews.apache.org/r/56884/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/56870/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56756/#review166080
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 20, 2017
/main/resources/ui/config/environment.js 411ee99
Diff: https://reviews.apache.org/r/56839/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56604/#review166055
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 13, 2017, 5
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56606/#review165860
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 16, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56756/#review165849
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 16, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56704/#review165810
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 15, 2017
/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56643/#review165518
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 14, 2017, 5
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56647/#review165479
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 14, 2017, 9
/src/main/resources/ui/app/templates/queries/query.hbs
5b2b5c2
Diff: https://reviews.apache.org/r/56488/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
://reviews.apache.org/r/56488/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
manually tested.
Thanks,
Pallav Kulshreshtha
blank lines.
contrib/views/hive20/src/main/resources/ui/app/routes/queries/query.js (line
536)
<https://reviews.apache.org/r/56606/#comment237202>
space
- Pallav Kulshreshtha
On Feb. 13, 2017, 6:52 p.m., venkat sairam wrote:
>
>
----
On Feb. 11, 2017, 10:26 a.m., Pallav Kulshreshtha wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56572/
>
in jobs, removed extra function in query service.
- Pallav
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56483/#review164891
--------
---
manuualy tested
Thanks,
Pallav Kulshreshtha
/components/visual-explain-detail.hbs
PRE-CREATION
contrib/views/hive20/src/main/resources/ui/app/templates/components/visual-explain.hbs
4238d43
Diff: https://reviews.apache.org/r/56572/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
://reviews.apache.org/r/56542/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56538/#review165090
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 10, 2017, 9
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56379/#review165085
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 10, 2017, 7
/resources/ui/app/templates/queries/query.hbs
8ee8e87
Diff: https://reviews.apache.org/r/56488/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
manuualy tested
Thanks,
Pallav Kulshreshtha
/services/query.js b484c74
contrib/views/hive20/src/main/resources/ui/app/styles/app.scss 4f73ef8
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
8ee8e87
Diff: https://reviews.apache.org/r/56483/diff/
Testing
---
manuualy tested
Thanks,
Pallav
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56430/#review164849
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 8, 2017, 6
://reviews.apache.org/r/56443/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
://reviews.apache.org/r/56443/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/views/hive20/src/main/resources/ui/app/templates/savedqueries.hbs
0ab6cfd
Diff: https://reviews.apache.org/r/56443/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/savedqueries.hbs
0ab6cfd
Diff: https://reviews.apache.org/r/56443/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56383/#review164663
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 7, 2017, 11
b776fd2
contrib/views/hive20/src/main/resources/ui/app/templates/udfs/new.hbs
PRE-CREATION
Diff: https://reviews.apache.org/r/56345/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
://reviews.apache.org/r/56384/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
b776fd2
contrib/views/hive20/src/main/resources/ui/app/templates/udfs/new.hbs
PRE-CREATION
Diff: https://reviews.apache.org/r/56345/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/resources/ui/app/templates/udfs/new.hbs
PRE-CREATION
Diff: https://reviews.apache.org/r/56345/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56168/#review163939
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Feb. 1, 2017, 1
-table.hbs
180c43d
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
22838a4
Diff: https://reviews.apache.org/r/56084/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
-table.hbs
4fe6f43
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
5cdc9b3
Diff: https://reviews.apache.org/r/56084/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/main/resources/ui/app/routes/queries/query.js
753b7f7
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
5cdc9b3
Diff: https://reviews.apache.org/r/56115/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/query.js
753b7f7
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
5cdc9b3
Diff: https://reviews.apache.org/r/56115/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/ui/app/templates/databases/database/tables/table/ddl.hbs
cbbcdd0
Diff: https://reviews.apache.org/r/56078/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/ui/app/templates/components/query-result-table.hbs
4fe6f43
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
5cdc9b3
Diff: https://reviews.apache.org/r/56078/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/templates/components/query-result-table.hbs
4fe6f43
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
5cdc9b3
Diff: https://reviews.apache.org/r/56084/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/table/ddl.hbs
cbbcdd0
Diff: https://reviews.apache.org/r/56078/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
-result-table.hbs
4fe6f43
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
5cdc9b3
Diff: https://reviews.apache.org/r/56075/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/src/main/resources/ui/app/templates/queries/query.hbs
5cdc9b3
Diff: https://reviews.apache.org/r/56075/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
,
Pallav Kulshreshtha
/hive20/src/main/resources/ui/app/routes/queries/query.js
2ecf967
Diff: https://reviews.apache.org/r/56031/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/hive20/src/main/resources/ui/app/styles/app.scss e178222
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
9e9e542
Diff: https://reviews.apache.org/r/56030/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56014/#review163262
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 27, 2017, 1
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55837/#review162906
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 23, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55836/#review162904
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 23, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55834/#review162625
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 23, 2017, 9
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55718/#review162623
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 20, 2017, 5
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55757/#review162600
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 20, 2017
/ui/app/templates/queries/query.hbs
2138356
Diff: https://reviews.apache.org/r/55615/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
/ui/app/templates/queries/query.hbs
2138356
Diff: https://reviews.apache.org/r/55615/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
Diff: https://reviews.apache.org/r/55615/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
4bda978
contrib/views/hive20/src/main/resources/ui/app/templates/components/save-hdfs.hbs
5b53102
contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs
120a045
Diff: https://reviews.apache.org/r/55573/diff/
Testing
---
manually tested.
Thanks,
Pallav
: https://reviews.apache.org/r/55501/diff/
Testing
---
manually tested
Thanks,
Pallav Kulshreshtha
: https://reviews.apache.org/r/55501/diff/
Testing
---
manually tested
Thanks,
Pallav Kulshreshtha
/savedqueries.hbs
b776fd2
Diff: https://reviews.apache.org/r/55501/diff/
Testing
---
manually tested
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55378/#review161072
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 10, 2017
/hdfs-directory-viewer/addon/components/directory-viewer.js
(line 120)
<https://reviews.apache.org/r/55371/#comment232313>
length is calculated multiple times. It can be done in a variable and then
make use of that variable.
- Pallav Kulshreshtha
On Jan. 10, 2017, 7:18 a.m., Padma P
wer.js
(line 102)
<https://reviews.apache.org/r/55371/#comment232312>
if/else condition can be changed to one liner code.
- Pallav Kulshreshtha
On Jan. 10, 2017, 7:18 a.m., Padma Priya N wrote:
>
> ---
> This is an automatic
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55338/#review161039
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 9, 2017, 11
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55336/#review161037
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 9, 2017, 9
/query.hbs 69fc657
Diff: https://reviews.apache.org/r/55340/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55167/#review160700
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 4, 2017, 9
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55206/#review160680
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 5, 2017, 12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55203/#review160577
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 5, 2017, 10
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55170/#review160572
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 4, 2017, 12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55133/#review160564
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 4, 2017, 7
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55164/#review160562
---
Ship it!
Ship It!
- Pallav Kulshreshtha
On Jan. 4, 2017, 8
://reviews.apache.org/r/55137/diff/
Testing
---
manually tested.
Thanks,
Pallav Kulshreshtha
101 - 200 of 318 matches
Mail list logo