selda icon indicating copy to clipboard operation
selda copied to clipboard

MySQL backend support

Open valderman opened this issue 8 years ago • 7 comments

Currently, a few things are missing to be able to support a MySQL backend:

  • Support for ? as a parameter placeholder.
  • Probably more.

This is probably a good time to restructure the pretty-printer to be explicitly parameterized over some configuration, instead of over different functions in different places.

valderman avatar May 23 '17 11:05 valderman

For now my only request is to a custom placeholder ; )

winterland1989 avatar May 23 '17 13:05 winterland1989

@winterland1989 93bd5d0b22968af48beba8f60bfeb3c072cbf2f6 moves the pretty-printer configuration into a single type, and adds support for custom placeholders.

valderman avatar May 25 '17 21:05 valderman

Nice! Would you mind keeping this issue open? I'll update info if i come across further problem.

winterland1989 avatar May 26 '17 03:05 winterland1989

Indeed, I won't feel comfortable closing this until we actually have MySQL up and working.

valderman avatar May 26 '17 09:05 valderman

I have started a mysql backend at: https://github.com/kuribas/selda it implements all the features for the backend, but there are some issues to be solved (~~bug in mysql-haskell~~, conversion issues, etc..)

kuribas avatar Nov 12 '18 21:11 kuribas

@kuribas Can you speak more specifically about what the bug is?

winterland1989 avatar Nov 13 '18 03:11 winterland1989

Sorry, it was a bug in my own code, when copy pasting your functions.

kuribas avatar Nov 13 '18 09:11 kuribas