[beeswax] HS2 principal should come from hive-site.xml

Review Request #3075 — Created May 20, 2013 and submitted May 20, 2013, 12:08 p.m.

Information
Owner:	abec
Repository:	old-hue-rw
Branch:
Bugs:
Depends On:
Reviewers
Groups:	hue
People:	enricoberti, romain

Description

commit 18188c2e753ca2badae0f622f8029e636aa04ce6
Author: Abraham Elmahrek <abraham@elmahrek.com>
Date:   Mon May 20 16:24:24 2013 -0700

    [beeswax] HS2 principal should come from hive-site.xml

:100644 100644 2cdb2cf... 32568dd... M	apps/beeswax/src/beeswax/server/dbms.py

Testing Done

Ran test fast.

Issues

1
0
3
0
All Issues: 4

Description	From	Last Updated
No need to unit-test the thing? ;)	romain	May 20, 2013, 12:02 p.m.
append comment? # Beeswaxd runs as 'hue'	romain	May 20, 2013, 12:02 p.m.
Can be removed I guess, or will should un-indent one time and print query_server instead	romain	May 20, 2013, 12:05 p.m.
query_server is a dict with the legend inside but it is cool	romain	May 20, 2013, 12:09 p.m.

Ship it!

apps/beeswax/src/beeswax/hive_site.py (Diff revision 1)
Show all issues
```
No need to unit-test the thing? ;)
```
apps/beeswax/src/beeswax/server/dbms.py (Diff revision 1)
Show all issues
```
append comment?
# Beeswaxd runs as 'hue'
```

apps/beeswax/src/beeswax/server/dbms.py (Diff revision 1)

Show all issues

Can be removed I guess, or will should un-indent one time and print query_server instead

Diff:

Revision 2 (+20 -1)

Show changes

	apps/beeswax/src/beeswax/hive_site.py
	apps/beeswax/src/beeswax/tests.py
	apps/beeswax/src/beeswax/server/dbms.py

Testing Done:

~		N/A
	~	Ran test fast.

Status: Closed (submitted)

Ship it! Fix it!

```
Awesome!
```
apps/beeswax/src/beeswax/server/dbms.py (Diff revision 2)
Show all issues
```
query_server is a dict with the legend inside but it is cool
```

You have a pending review.

Review Board 3.0.18

[beeswax] HS2 principal should come from hive-site.xml

Diff:

Testing Done:

Status: Closed (submitted)