Ah, great. Thanks for clarifying that for me.
On Tue, Feb 21, 2017 at 11:44 AM, Daan Hoogland <daan.hoogl...@gmail.com> wrote: > No Rafael the other way around, > > The first time you call it it might change things, but no matter how > often you call it it will have changed in exactly the same way. > > On Tue, Feb 21, 2017 at 5:41 PM, Rafael Weingärtner > <rafaelweingart...@gmail.com> wrote: > > I think this might be others doubt as well. Sorry if it seems a > naïve/silly > > question. > > > > By idempotent, I understand something that you can make as many > operations > > as possible and it does not change (broadly speaking). For example, (in > > theory), when you do an HTTP Get requests, the response would be always > the > > same, and the state of the resource should not change. > > > > Now, regarding SQLs; this SQL for instance “CREATE TABLE XXX…..” is > > idempotent for you (as far as I understood reading this thread), right? > > > > And something like “CREATE or REPLACE TABLE XXX…..” would be > > non-idempotent. Did I understand it right? > > > > On Tue, Feb 21, 2017 at 11:28 AM, Daan Hoogland <daan.hoogl...@gmail.com > > > > wrote: > > > >> On Tue, Feb 21, 2017 at 3:19 PM, Marc-Aurèle Brothier < > ma...@exoscale.ch> > >> wrote: > >> > > >> > Daan, the project maintainers should enforce that. I also posted > another > >> > finding that the upgrade path are not identical due to the order in > which > >> > upgrade files are executed, see (https://github.com/apache/ > >> > cloudstack/pull/1768) > >> > >> If you mean refuse PRs containing non-idem-potent sql code yes, but as > >> for real work it is all on a voluntary basis, that is someone must > >> find it worth the time to encode it. I complete agree with a policy to > >> refuse comntaining other creates and drop then as in > >> > >> > > "CREATE OR REPLACE VIEW..." "DROP IF EXISTS....". > >> > >> So please feel free to speak up if you catch somebody trying to sneak > >> in code like that. They have my -1 > >> > >> -- > >> Daan > >> > > > > > > > > -- > > Rafael Weingärtner > > > > -- > Daan > -- Rafael Weingärtner
