[GitHub] [arrow-ballista] DaltonModlin commented on pull request #442: Add optional flag which advertises host for Arrow Flight SQL #418

[GitBox <gi...@apache.org>]

DaltonModlin commented on PR #442:
URL: https://github.com/apache/arrow-ballista/pull/442#issuecomment-1289600567

   I've validated my changes have the expected outcome via writing a test in the flight portion of the arrow repo.
   ```
   @Test
   public void testBallistaFlag() throws Exception {
       String url = "jdbc:arrow-flight://127.0.0.0:50050";
       java.util.Properties props = new java.util.Properties();
       props.setProperty("useEncryption", "false");
       props.setProperty("user", "admin");
       props.setProperty("password", "password");
       Connection con = DriverManager.getConnection(url, props);
       // statement
       java.sql.Statement stmt = con.createStatement();
       String sql = "create external table customer STORED AS CSV WITH HEADER ROW LOCATION 'arrow-datafusion/datafusion/core/tests/tpch-csv/customer.csv';\n";
       assertTrue(stmt.execute(sql));
   }
   ```
   Running this test in conjunction with a local scheduler (having passing it `--advertise-host="127.0.0.2:50050"`) and a local executor shows the request for flight results is proxied via the Scheduler rather than going directly to the executor IP.


-- 
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: github-unsubscribe@arrow.apache.org

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