You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by "jongyoul (via GitHub)" <gi...@apache.org> on 2023/06/15 06:28:09 UTC

[GitHub] [zeppelin] jongyoul opened a new pull request, #4617: [HOTFIX] Add bank.csv for integration test

jongyoul opened a new pull request, #4617:
URL: https://github.com/apache/zeppelin/pull/4617

   ### What is this PR for?
   Fixing errors for integration test. The file was being served as s3 endpoint but it's not accessible for now
   
   
   ### What type of PR is it?
   Hot Fix
   
   ### Todos
   * [x] - Add the file which will be pointed as URL for the next PR to fix integration test
   
   ### What is the Jira issue?
   N/A
   
   ### How should this be tested?
   In the next PR, I'll change the URL from `https://s3.amazonaws.com/apache-zeppelin/tutorial/bank/bank.csv`  to `https://raw.githubusercontent.com/apache/zeppelin/master/testing/resources/bank.csv`
   
   ### Screenshots (if appropriate)
   ```
   java.io.IOException: Server returned HTTP response code: 403 for URL: https://s3.amazonaws.com/apache-zeppelin/tutorial/bank/bank.csv
     at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1902)
     at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1500)
     at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:268)
     at java.net.URL.openStream(URL.java:1093)
   ```
   
   ### Questions:
   * Does the license files need to update? No
   * Is there breaking changes for older versions? No 
   * Does this needs documentation? No
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [zeppelin] Reamer merged pull request #4617: [HOTFIX] Add bank.csv for integration test

Posted by "Reamer (via GitHub)" <gi...@apache.org>.
Reamer merged PR #4617:
URL: https://github.com/apache/zeppelin/pull/4617


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [zeppelin] jongyoul commented on pull request #4617: [HOTFIX] Add bank.csv for integration test

Posted by "jongyoul (via GitHub)" <gi...@apache.org>.
jongyoul commented on PR #4617:
URL: https://github.com/apache/zeppelin/pull/4617#issuecomment-1596371748

   @Reamer @zjffdu @huage1994 Could you please approve this PR? :-)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org