jizezhang commented on PR #19388:
URL: https://github.com/apache/datafusion/pull/19388#issuecomment-3703906654

   > Thanks @jizezhang -- this code looks very nice
   > 
   > I was testing it out, but it seems to now show anything after creating an 
external table:
   > 
   > ```sql
   > DataFusion CLI v51.0.0
   > > SELECT metadata_size_bytes, expires_in, metadata_list FROM 
list_files_cache();
   > +---------------------+------------+---------------+
   > | metadata_size_bytes | expires_in | metadata_list |
   > +---------------------+------------+---------------+
   > +---------------------+------------+---------------+
   > 0 row(s) fetched.
   > Elapsed 0.021 seconds.
   > 
   > > CREATE EXTERNAL TABLE nyc_taxi_rides
   > STORED AS PARQUET LOCATION 
's3://altinity-clickhouse-data/nyc_taxi_rides/data/tripdata_parquet/'
   > ;
   > 0 row(s) fetched.
   > Elapsed 8.026 seconds.
   > 
   > > SELECT metadata_size_bytes, expires_in, metadata_list FROM 
list_files_cache();
   > +---------------------+------------+---------------+
   > | metadata_size_bytes | expires_in | metadata_list |
   > +---------------------+------------+---------------+
   > +---------------------+------------+---------------+
   > 0 row(s) fetched.
   > Elapsed 0.007 seconds.
   > ```
   > 
   > I would expect the second command to show the contents of the cache
   > 
   > Maybe we just need to merge up and pick up the PR from @BlakeOrth to add a 
default
   > 
   >     * #19366
   
   Yes just merged upstream.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to