Fix alluxio path replacement regex to handle s3a:// and s3n:// urls (#84

) * replace s3a s3n * fix * fix * fix * update changelog. * update regex Co-authored-by: Raj Poluri <[email protected]>
ExpediaGroup · Mar 30, 2021 · 94f5bc9 · 94f5bc9
1 parent 2da3d43
commit 94f5bc9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,10 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [3.3.2] - 2021-03-30
+### Added
+- Fix alluxio path replacement regex to handle s3a:// and s3n:// urls.
+
 ## [3.3.1] - 2021-03-30
 ### Added
 - Variable to configure waggle-dance log level.

diff --git a/hive-site.tf b/hive-site.tf
@@ -11,7 +11,7 @@ data "template_file" "hive_site_xml" {
 %{for s3_bucket in split(",", alluxio_endpoint.s3_buckets)}
    <property>
        <name>apiary.path.replacement.regex.alluxio-${s3_bucket}</name>
-       <value>^(s3://)${s3_bucket}/.*</value>
+       <value>^(s3[an]?://)${s3_bucket}/.*$</value>
    </property>
    <property>
        <name>apiary.path.replacement.value.alluxio-${s3_bucket}</name>