531651225 commented on PR #2499:
URL: 
https://github.com/apache/incubator-seatunnel/pull/2499#issuecomment-1229037835

   > > > > > Can phoenix drivers be loaded using jdbc connector?
   > > > > 
   > > > > 
   > > > > Can we still use this method ? Considering that more improvements 
will be added later, is this more flexible and less conflict
   > > > 
   > > > 
   > > > case #2429
   > > 
   > > 
   > > Reuse JDBC connectors. Does the Phoenix connector need to be added to 
the connector JDBC module
   > 
   > You need add database dialect to here
   > 
   > 
https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/internal/dialect
   
   
   
   > > > > > Can phoenix drivers be loaded using jdbc connector?
   > > > > 
   > > > > 
   > > > > Can we still use this method ? Considering that more improvements 
will be added later, is this more flexible and less conflict
   > > > 
   > > > 
   > > > case #2429
   > > 
   > > 
   > > Reuse JDBC connectors. Does the Phoenix connector need to be added to 
the connector JDBC module
   > 
   > You need add database dialect to here
   > 
   > 
https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/internal/dialect
   
   Thanks for your advice, I have re implemented the phoenix connector reuse  
jdbc connector in new commit. and and spark and flink e2e  and pass the test


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to