HUE-8364 [s3] Fix s3 not getting initialized in certain scenarios

Review Request #13054 - Created June 8, 2018 and submitted

Jean-Francois Desjeans Gauthier
hue
master
HUE-8364
hue
jgauthier
commit 710c2c1fe495199fc25b54fa8e8870b3478cd0fb
Author: jdesjean <jgauthier@cloudera.com>
Date:   Fri Jun 8 16:40:47 2018 -0700

    HUE-8364 [s3] Fix s3 not getting initialized in certain scenarios

:100644 100644 0e5785c351... 22c079ed64... M	desktop/libs/aws/src/aws/conf.py


  • 1
  • 0
  • 1
  • 1
  • 3
Description From Last Updated
BTW, no way to ge this list dynamically via Boto at some point? (seems it will always bound to fail ... Romain Rigaux
  1. Ship it! \,,/(^_^)\,,/

  2. desktop/libs/aws/src/aws/conf.py (Diff revision 2)
     
     

    there should be also an eu-west-3 now (paris if i recall correctly) and another one in China

  3. 
      
  1. 
      
  2. desktop/libs/aws/src/aws/conf.py (Diff revision 2)
     
     

    BTW, no way to ge this list dynamically via Boto at some point? (seems it will always bound to fail sooner or later)

    https://docs.aws.amazon.com/general/latest/gr/rande.html
    https://docs.aws.amazon.com/cli/latest/reference/ec2/describe-regions.html

    1. It's not available to get dynamically from the library and since this is done at initialization, not sure it's worth a network call to get it.
      @Romain, should we drop the check to validate the region? We don't use that information in any other way.

  3. desktop/libs/aws/src/aws/s3/s3fs.py (Diff revision 2)
     
     

    Why do we stop using 'Location' in general now?

    1. Recently, I can no longer access the properties. Prakash mentions there's probably a conflict that prevent the properties from getting resolved. I did not investigate further.

  4. 
      
Review request changed

Status: Closed (submitted)

Loading...