HADOOP-17714 ABFS: testBlobBackCompatibility, testRandomRead & WasbAbfsCompatibility tests fail when triggered with default configs (#3035)
This commit is contained in:
parent
5970c632d4
commit
35e4c31fff
|
@ -149,7 +149,7 @@ public abstract class AbstractAbfsIntegrationTest extends
|
|||
if (!isIPAddress
|
||||
&& (abfsConfig.getAuthType(accountName) != AuthType.SAS)
|
||||
&& !abfs.getIsNamespaceEnabled()) {
|
||||
final URI wasbUri = new URI(abfsUrlToWasbUrl(getTestUrl()));
|
||||
final URI wasbUri = new URI(abfsUrlToWasbUrl(getTestUrl(), abfsConfig.isHttpsAlwaysUsed()));
|
||||
final AzureNativeFileSystemStore azureNativeFileSystemStore =
|
||||
new AzureNativeFileSystemStore();
|
||||
|
||||
|
@ -355,13 +355,13 @@ public abstract class AbstractAbfsIntegrationTest extends
|
|||
protected static String wasbUrlToAbfsUrl(final String wasbUrl) {
|
||||
return convertTestUrls(
|
||||
wasbUrl, FileSystemUriSchemes.WASB_SCHEME, FileSystemUriSchemes.WASB_SECURE_SCHEME, FileSystemUriSchemes.WASB_DNS_PREFIX,
|
||||
FileSystemUriSchemes.ABFS_SCHEME, FileSystemUriSchemes.ABFS_SECURE_SCHEME, FileSystemUriSchemes.ABFS_DNS_PREFIX);
|
||||
FileSystemUriSchemes.ABFS_SCHEME, FileSystemUriSchemes.ABFS_SECURE_SCHEME, FileSystemUriSchemes.ABFS_DNS_PREFIX, false);
|
||||
}
|
||||
|
||||
protected static String abfsUrlToWasbUrl(final String abfsUrl) {
|
||||
protected static String abfsUrlToWasbUrl(final String abfsUrl, final boolean isAlwaysHttpsUsed) {
|
||||
return convertTestUrls(
|
||||
abfsUrl, FileSystemUriSchemes.ABFS_SCHEME, FileSystemUriSchemes.ABFS_SECURE_SCHEME, FileSystemUriSchemes.ABFS_DNS_PREFIX,
|
||||
FileSystemUriSchemes.WASB_SCHEME, FileSystemUriSchemes.WASB_SECURE_SCHEME, FileSystemUriSchemes.WASB_DNS_PREFIX);
|
||||
FileSystemUriSchemes.WASB_SCHEME, FileSystemUriSchemes.WASB_SECURE_SCHEME, FileSystemUriSchemes.WASB_DNS_PREFIX, isAlwaysHttpsUsed);
|
||||
}
|
||||
|
||||
private static String convertTestUrls(
|
||||
|
@ -371,15 +371,17 @@ public abstract class AbstractAbfsIntegrationTest extends
|
|||
final String fromDnsPrefix,
|
||||
final String toNonSecureScheme,
|
||||
final String toSecureScheme,
|
||||
final String toDnsPrefix) {
|
||||
final String toDnsPrefix,
|
||||
final boolean isAlwaysHttpsUsed) {
|
||||
String data = null;
|
||||
if (url.startsWith(fromNonSecureScheme + "://")) {
|
||||
if (url.startsWith(fromNonSecureScheme + "://") && isAlwaysHttpsUsed) {
|
||||
data = url.replace(fromNonSecureScheme + "://", toSecureScheme + "://");
|
||||
} else if (url.startsWith(fromNonSecureScheme + "://")) {
|
||||
data = url.replace(fromNonSecureScheme + "://", toNonSecureScheme + "://");
|
||||
} else if (url.startsWith(fromSecureScheme + "://")) {
|
||||
data = url.replace(fromSecureScheme + "://", toSecureScheme + "://");
|
||||
}
|
||||
|
||||
|
||||
if (data != null) {
|
||||
data = data.replace("." + fromDnsPrefix + ".",
|
||||
"." + toDnsPrefix + ".");
|
||||
|
|
|
@ -74,6 +74,12 @@ public class ITestAzureBlobFileSystemBackCompat extends
|
|||
+ ";AccountName=" + this.getAccountName().split("\\.")[0]
|
||||
+ ";AccountKey=" + this.getAccountKey();
|
||||
}
|
||||
else if (this.getConfiguration().isHttpsAlwaysUsed()) {
|
||||
connectionString = "DefaultEndpointsProtocol=https;BlobEndpoint=https://"
|
||||
+ this.getAccountName().replaceFirst("\\.dfs\\.", ".blob.")
|
||||
+ ";AccountName=" + this.getAccountName().split("\\.")[0]
|
||||
+ ";AccountKey=" + this.getAccountKey();
|
||||
}
|
||||
else {
|
||||
connectionString = "DefaultEndpointsProtocol=http;BlobEndpoint=http://"
|
||||
+ this.getAccountName().replaceFirst("\\.dfs\\.", ".blob.")
|
||||
|
|
|
@ -159,7 +159,7 @@ public class ITestWasbAbfsCompatibility extends AbstractAbfsIntegrationTest {
|
|||
String abfsUrl = "abfs://abcde-1111-1111-1111-1111@xxxx.dfs.xxx.xxx.xxxx.xxxx";
|
||||
String wabsUrl = "wasb://abcde-1111-1111-1111-1111@xxxx.blob.xxx.xxx.xxxx.xxxx";
|
||||
assertEquals(abfsUrl, wasbUrlToAbfsUrl(wabsUrl));
|
||||
assertEquals(wabsUrl, abfsUrlToWasbUrl(abfsUrl));
|
||||
assertEquals(wabsUrl, abfsUrlToWasbUrl(abfsUrl, false));
|
||||
}
|
||||
|
||||
@Test
|
||||
|
|
Loading…
Reference in New Issue