Re: Supporting non-free SQL clients in ob-sql (was: [PATCH] ob-sql: Add support for Athena)

[Ihor Radchenko]

Tim Cross <theophilusx@gmail.com> writes:

> 3. There is no requirement to install non-free software to use
> ob-sql.el. The software is fully functional using a free RDMS like
> postgres.

Yes, but there is requirement to install in order to use ob-sql.el
__with :engine set to non-free option__.

So, I can envision that someone who decided to use ob-sql.el and
considering between free and non-free engine may prefer non-free one. Of
course, it is not very strong argument, but the boundaries are fuzzy in
this area.

> For maintenance reasons and to add session support, I would suggest that
> using sql.el instead of re-inventing this wheel would be a better
> outcome. I've used sql.el for years and it works extremely well and I
> don't htink it would be too hard to integrate into ob-sql.

Sure. That's what we usually do - just use whatever REPL is available
for a given ob-* language. Just a question of someone sending a patch.

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>