HUE-8407 [fb] Overwriting default_hdfs_superuser in configuration does not take effect

Review Request #13141 — Created July 2, 2018 and submitted July 24, 2018, 2:31 p.m.

Information
Owner:	roohi
Repository:	hue
Branch:	HUE-8403_pig_jd_hue3
Bugs:	HUE-8407
Depends On:
Reviewers
Groups:	hue
People:	enricoberti, jgauthier, johan, ranade, romain, weixia, yingc

Description

commit 57daef90dc281bac7de923fcdf6f0e3e42f8f470
Author: Roohi <roohisyeda@cloudera.com>
Date:   Mon Jul 2 15:19:28 2018 -0700

    HUE-8407 [fb] Overwriting default_hdfs_superuser in configuration does not take effect

:100644 100644 9d577ffa4a... 85eac8b92d... M	desktop/libs/hadoop/src/hadoop/fs/webhdfs.py

Testing Done

Manual

Issues

1
0
0
1
All Issues: 2

Description	From	Last Updated
Can we always set default_hdfs_superuser as superuser ignoring the stats?	roohi	July 2, 2018, 3:22 p.m.
Why not desktop.conf.DEFAULT_HDFS_SUPERUSER.get()?	jgauthier	July 3, 2018, 12:17 p.m.

Fix it!

Show all issues

Can we always set default_hdfs_superuser as superuser ignoring the stats?

jgauthier July 3, 2018, 12:18 p.m.

I think offering this option in the configuration is weird. That being said, I think since we offer the option, the logic you're using seems what is expected.

desktop/libs/hadoop/src/hadoop/fs/webhdfs.py (Diff revision 1)
Show all issues
```
Why not desktop.conf.DEFAULT_HDFS_SUPERUSER.get()?
```

Ship it!

```
Ship It!
```

Ship it!

```
Ship It!
```

You have a pending review.

Review Board 3.0.18

HUE-8407 [fb] Overwriting default_hdfs_superuser in configuration does not take effect

Status: Closed (submitted)