---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23820/#review48483
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23820/
---
(Updated July 23, 2014, 12:38 p.m.)
Review request for hive, Brock Noland,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23820/#review48601
---
Ship it!
- Szehon Ho
On July 23, 2014, 12:38 p.m., Chaoyu Tang
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23820/
---
Review request for hive, Brock Noland, Prasad Mujumdar, Szehon Ho, and Xuefu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23820/#review48441
---
Hey Chaoyu thanks, the logic looks good now. Can you also fix all
On July 22, 2014, 10:33 p.m., Szehon Ho wrote:
Hey Chaoyu thanks, the logic looks good now. Can you also fix all the
red-space? It's two lines per indent in hive, thanks.
Just realized I had a typo, its two spaces per indent.
- Szehon
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23820/
---
(Updated July 23, 2014, 2:27 a.m.)
Review request for hive, Brock Noland,
On July 22, 2014, 10:33 p.m., Szehon Ho wrote:
ql/src/java/org/apache/hadoop/hive/ql/lockmgr/zookeeper/ZooKeeperHiveLockManager.java,
line 300
https://reviews.apache.org/r/23820/diff/1/?file=639471#file639471line300
Why was this necessary to change?
Hey, Szehon, Thanks for the