[Tarantool-patches] [PATCH] box: remove unicode_ci for functions

Kirill Yukhin kyukhin at tarantool.org
Tue Dec 10 15:44:21 MSK 2019


Hello,

On 10 дек 11:19, Konstantin Osipov wrote:
> * Vladislav Shpilevoy <v.shpilevoy at tarantool.org> [19/12/10 10:22]:
> > >> We are team of Tarantool developers working for MailRU Group.
> > > 
> > > Actually this decision was made without any discussions - it was
> > > quickly hacked in back in 2018. Maybe you and Nikita had some
> > > discussion prompted by Peter's firm stance that we should
> > > uppercase, but that was it. I protested several times while I
> > > was still on board but had not anticipated how bad the solution
> > > would turn out to be when it is implemented to follow through on
> > > time. Now it is still not to late - bit it's getting more late
> > > every day.
> > > 
> > > What we learned since then is that every single newbie trips over
> > > it.
> > > 
> > > 
> > 
> > Well, you was exactly the person who voted for uppercase of
> > everything. Peter was for it too, but you made the decision,
> > and you at that time had a right to forbid it.
> 
> I agree I am partly to blame because I had a chance to object 
> and I didn't do it strongly enough at the time. 
> In any case, I changed my mind seeing how it worked out.

This patch is not about matter which is being discussed in this
thread. To discuss such major features (like `lets do everything
CI`) we have `discussions` mailing list and github issues.
I recommend you to move there with such proposals.

The patch LGTM. I've checked it into master.

--
Regards, Kirill Yukhin


More information about the Tarantool-patches mailing list