turbaszek commented on a change in pull request #13847:
URL: https://github.com/apache/airflow/pull/13847#discussion_r563263804



##########
File path: airflow/providers/exasol/hooks/exasol.py
##########
@@ -108,6 +108,37 @@ def get_first(self, sql: Union[str, list], parameters: 
Optional[dict] = None) ->
             with closing(conn.execute(sql, parameters)) as cur:
                 return cur.fetchone()
 
+    def export_to_file(

Review comment:
       @XD-DENG is right - I hoped that using the `Pyexasol` method will give 
us most flexibility and will be easiest to maintain. Here is the exasol info:
   
   
   >The target file can either be CSV or FBV files and should comply to the 
format specifications in the CSV Data Format and the Fixblock Data format 
(FBV). File names may only consist of ASCII characters. A BOMClosed is not 
supported.
   
   https://docs.exasol.com/sql/export.htm#UsageNotes
   
   Supporting formats not supported by exasol is something that we may consider 
adding using pandas (read_csv -> df -> save_as_format). However, I'm not sure 
if this is something that should happen in Airflow task. The data can be 
arbitrary big so keeping it in memory may be problematic.




----------------------------------------------------------------
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.

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


Reply via email to