display improvements to %sqlcmd snippets

[edublancas]

we need to do some improvements when showing feedback to the user

for context: snippets.ipynb.zip

[bbeat2782]

If you have nothing in mind what I should work on next, I can start working on this.

Acceptance Criteria

1. Calling %sqlcmd snippets when there is a no snippet prints out No snippets stored message using the display module.
2. Calling %sqlcmd snippets when there are stored snippets displays table and snippet for all stored snippets.
3. Modify %sqlcmd snippets documentation to reflect this change
4. Modify test functions

[edublancas]

sure, please work on this

[bbeat2782]

@edublancas
I have one question about the implementation of this issue.

Let's say a user connects to connection A, creates a snippet under connection A, and then connects to another connection B. In this case, is the expected behavior of calling %sqlcmd snippets when in connection B printing out No snippets stored since no snippets were created under connection B?

[edublancas]

interesting point. right now, snippets are connection-independent so switching connections won't have any effect. they'll still show the same ones. But I like this Idea, we can work on it later.

[bbeat2782]

Sorry for not providing a clear explanation in my first question. @edublancas

You are right about snippets being connection-independent. So there is no problem with getting a stored snippet from different connections. The problem occurs when I try to display a result of a snippet when the snippet uses a table that is not present in the current connection.

The following code is a short demonstration. The full notebook is here
(connection_change_demonstration.ipynb.zip).

After
%sql duckdb://
%%sql --save gentoo SELECT * FROM penguins.csv where species == 'Gentoo'

%sqlcmd snippets gentoo returns the stored snippet and %sql SELECT * FROM penguins.csv where species == 'Gentoo' returns a result of the snippet.

After changing the connection with
%sql sqlite://

%sqlcmd snippets gentoo still returns the stored snippet but %sql SELECT * FROM penguins.csv where species == 'Gentoo' does not return the same result because of (sqlite3.OperationalError) no such table: penguins.csv error.

Since the objective is to show a table and snippet when %sqlcmd snippets is called but some snippets are not runnable under some connections, should I only include snippets for snippets that are using tables not present in the current connection or should I try different connections to display tables for all snippets?

[edublancas]

thanks for providing more feedback. I think let's keep it simple and allow users to run any snippets on any connection, it's on their side to ensure that whatever snippet they want to run is valid in the current connection