HUE-703 [beeswax] Saving the result of a query raises an exception

Review Request #2043 — Created April 24, 2012 and submitted

romain
old-hue-rw
HUE-703
hue
bcwalrus
commit a5ff7c59e2a9ac2563c98d723077597540917491
Author: Romain Rigaux <romain@cloudera.com>
Date:   Tue Apr 24 10:34:18 2012 -0700

    HUE-703 [beeswax] Saving the result of a query raises an exception

:100644 100644 8c8b666... 5c25645... M  apps/beeswax/src/beeswax/views.py
tests ran
manually saved a result
bcwalrus
  1. For the beeswax use case, can it just call `rename(old, new)' to move the entire directory?
    1. Yes it works, I was just worried to break something. Are there any reason for keeping the Hive output folder?
    2. Don't think so. Although we do need to worry about permissions on that folder when moving it -- whether we have permission in the first place, and what the resulting permission should be.
    3. Ok, it will be cleaner.
      
      An regarding the permissions it should be fine:
      The directory is moved for example from /tmp/hive-beeswax-romain/hive_2012-04-09_10-05-47_365_3830180043734098122 (/tmp/hive-beeswax-romain belongs to romain) to the path specified by romain and the same permissions are kept (655).
  2. 
      
romain
bcwalrus
  1. Ship It!
  2. 
      
romain
Review request changed

Status: Closed (submitted)

Loading...