>From Wael Alkowaileet <[email protected]>:

Wael Alkowaileet has submitted this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/13304 )

Change subject: [NO ISSUE] Remove unused function from Parquet Azure
......................................................................

[NO ISSUE] Remove unused function from Parquet Azure

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
Remove emulator endpoint extraction method

Change-Id: I67a2459a22e3802eed8dc000f56dd7e941543295
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/13304
Reviewed-by: Hussain Towaileb <[email protected]>
Integration-Tests: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>
---
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/record/reader/azure/parquet/AzureBlobParquetReaderFactory.java
1 file changed, 0 insertions(+), 5 deletions(-)

Approvals:
  Hussain Towaileb: Looks good to me, approved
  Jenkins: Verified; Verified

Objections:
  Anon. E. Moose #1000171: Violations found



diff --git 
a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/record/reader/azure/parquet/AzureBlobParquetReaderFactory.java
 
b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/record/reader/azure/parquet/AzureBlobParquetReaderFactory.java
index 4d1311c..b2b667a 100644
--- 
a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/record/reader/azure/parquet/AzureBlobParquetReaderFactory.java
+++ 
b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/record/reader/azure/parquet/AzureBlobParquetReaderFactory.java
@@ -116,11 +116,6 @@
         return uri.substring(uri.indexOf("//") + "//".length());
     }

-    private String extractEndPointForEmulator(String uri) {
-        String emulatorURI = extractEndPoint(uri);
-        return emulatorURI.substring(0, emulatorURI.indexOf('/'));
-    }
-
     private static void appendFileURI(StringBuilder builder, String container, 
String endPoint, BlobItem file) {
         
builder.append(ExternalDataConstants.AzureBlob.HADOOP_AZURE_BLOB_PROTOCOL);
         builder.append("://");

--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/13304
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I67a2459a22e3802eed8dc000f56dd7e941543295
Gerrit-Change-Number: 13304
Gerrit-PatchSet: 2
Gerrit-Owner: Wael Alkowaileet <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Wael Alkowaileet <[email protected]>
Gerrit-MessageType: merged

Reply via email to