[Bug 1457929] Review Request: proxysql, a high-performance MySQL proxy

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1457929



--- Comment #16 from Augusto Caringi <acaringi@xxxxxxxxxx> ---
(In reply to René Cannaò from comment #13)
> Hi all,
> 
> Author of ProxySQL here.
> First of all, thank you for considering including ProxySQL in Fedora
> official repository.
> I will share my thoughts.

René, thank you very much for joining this discussion, your opinion is very
important here.

> About [1] :
> my vote goes for 1.3 as well, to create a new user.

Good to know.

> About [2]:
> support for ARM should be as simple as removing all spinlock (this is where
> assembly code is used) and replace them with pthread rwlock. It is on my
> todo list.

That's nice, I've seen your latest commits removing spinlocks.

> About the dependencies currently bundled, the easy to de-bundle are
> libconfig and libdamon.
> SQLite3 is also patched, and there are features in the roadmap that require
> SQLite 3.15 , for example https://github.com/sysown/proxysql/issues/1011

But the latest stable version of ProxySQL (1.3.7) has a unpatched SQLite,
right?

I think that we can de-bundle it by now, and in upcoming versions see what's
the best approach.

Fedora 26 will ship with SQLite 3.19. Would be good if we could be able to
upstream this 'from_unixtime' patch
(https://github.com/sysown/proxysql/blob/master/deps/sqlite3/from_unixtime.patch)
and keep using a de-bundled SQLite in the future. What do you think?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux