HUE-9062 [beeswax] Add http thrift port configuration

Review Request #14434 — Created Nov. 1, 2019 and submitted

jgauthier
hue
master
HUE-9062
hue
jgauthier, johan, ranade, romain, weixia, yingc
commit 5b06ff9a54c629cd440cc7bf0f252a2d3ecd01e2
Author: Jean-Francois Desjeans Gauthier <jf.desjeans.gauthier@gmail.com>
Date:   Fri Nov 1 11:53:55 2019 -0700

    HUE-9062 [beeswax] Add http thrift port configuration
    Change-Id: I295768344ffcdac5c3db1cef759cfb10da56ca5f

:100644 100644 c749f2da31... 17793acb57... M	apps/beeswax/src/beeswax/conf.py
:100644 100644 9d044da506... 70dcb0ea25... M	apps/beeswax/src/beeswax/hive_site.py
:100644 100644 245419dd0a... d19dc173db... M	apps/beeswax/src/beeswax/server/dbms.py
:100644 100644 22b3d17979... 57df46ba37... M	desktop/conf.dist/hue.ini
:100644 100644 240f0b5fc7... 056e150572... M	desktop/conf/pseudo-distributed.ini.tmpl


  • 0
  • 0
  • 3
  • 0
  • 3
Description From Last Updated
romain
  1. Nice!

    No need to update https://github.com/cloudera/hue/blob/master/apps/beeswax/src/beeswax/server/dbms_tests.py#L68?

  2. apps/beeswax/src/beeswax/conf.py (Diff revision 1)
     
     

    nits: thrift --> Thrift, http --> HTTP or Http

  3. apps/beeswax/src/beeswax/conf.py (Diff revision 1)
     
     

    nit: extra space after 'llap'

  4. apps/beeswax/src/beeswax/hive_site.py (Diff revision 1)
     
     
    nit: somehow protect with lower case?
    
    (_CNF_HIVE_EXECUTION_MODE, 'tez').lower()
  5. 
      
jgauthier
jgauthier
Review request changed

Status: Closed (submitted)

Loading...