Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flightsql connection working #104

Merged
merged 5 commits into from
Aug 27, 2024

Conversation

matthewmturner
Copy link
Collaborator

@matthewmturner matthewmturner commented Aug 26, 2024

Adds a new tab for FlightSQL client

image

@matthewmturner matthewmturner requested a review from alamb August 26, 2024 16:02
@alamb
Copy link
Contributor

alamb commented Aug 27, 2024

I played around a bit with the code and the ui -- it took me some time getting used to it (kind of vim-like bindings) but once I got the hang of it, it was quite cool

Some feedback is:

  1. I found it very hard to figure out how to quit (especially once I got into the logs pane) -- I tried Ctrl-C, Ctrl-D, and q which did not work
  2. I also kept trying to want to get some sort of help by typing h or ? but neither of those keys did what I expected

@alamb
Copy link
Contributor

alamb commented Aug 27, 2024

I fixed some clippy issues and I think we should just merge this as is

@alamb alamb merged commit 1b9211c into datafusion-contrib:main Aug 27, 2024
3 checks passed
@matthewmturner
Copy link
Collaborator Author

ah i jsut got this completely working this morning, was about to push up some more changes. but ill just do another pr.

but i agree on the point regarding quiting and keys in general. im still giving that some thought.

@matthewmturner
Copy link
Collaborator Author

@alamb it also has some key handling improvements.

@alamb
Copy link
Contributor

alamb commented Aug 27, 2024

Cool -- sorry if I messed you up -- I am thinking given this is an early and fast moving project we can just merge quickly

I am working on adding

  1. a -f command to run multiple commands from a file
  2. Some way to write tests with the context (and maybe even a persistent catalog 🤔 )

I am thinking that way I can focus on the code that sets up / runs queries as you are working on the UI

@matthewmturner
Copy link
Collaborator Author

@alamb no problem at all. its definitely very fast moving, i think the next pr im about to do has some pretty good improvements and flightsql is fully working now.

regarding point 1 did you try setting ddl in ~/.datafusion/.datafusionrc

there is logic here for that which iterates over all queries in it

@alamb
Copy link
Contributor

alamb commented Aug 27, 2024

@alamb no problem at all. its definitely very fast moving, i think the next pr im about to do has some pretty good improvements and flightsql is fully working now.

regarding point 1 did you try setting ddl in ~/.datafusion/.datafusionrc

I did not

there is logic here for that which iterates over all queries in it

Nice

What I am (not so) secretly hoping, is to create some sort of file based catalog format (similar to the duckdb .db format) that can have DDL but also more crazy things like the contents of parquet metadata from remote object stores

But I don't even really have time to do what I want today, let alone this other cool stuff. Some day

@matthewmturner
Copy link
Collaborator Author

@alamb actually after seeing the diff i think the key handling improvements were already there. but i just merged my latest changes. the next thing i wanted to get working was an s3 feature so i could have DDL that uses that.

also just making sure you saw - there is some help on the bottom of the screen that says some key bindings

@matthewmturner
Copy link
Collaborator Author

What I am (not so) secretly hoping, is to create some sort of file based catalog format (similar to the duckdb .db format) that can have DDL but also more crazy things like the contents of parquet metadata from remote object stores

That would be very cool and i think could bring a lot of value to the ecosystem. did you have any preliminary thoughts on the metadata part?

we could definitely add an experimental feature for exploring that topic.

i do also plan on adding some catalog / schema exploration capabilities so could be related to that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants