From: | Hiroshi Inoue <Inoue(at)tpf(dot)co(dot)jp> |
---|---|
To: | Vadim Mikheev <vadim(at)krs(dot)ru> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Developers List <hackers(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] Re: [GENERAL] drop/rename table and transactions |
Date: | 1999-12-01 12:24:29 |
Message-ID: | 3845137D.8ABC6BDD@tpf.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general pgsql-hackers |
Vadim Mikheev wrote:
> Hiroshi Inoue wrote:
> >
> > > >
> > > > If there's no objection,I would change UnlockRelation() to not release
> > > > the specified lock except AccessShareLock.
> > >
> > > Why don't remove this call from improper places?
> > > I would try to find all calls and understand why
> > > they made...
> > >
> >
> > I was surprized that few people really want DDL commands inside transactions.
> > Are there any reasons to releasing lock before end of transaction except
> > that long term lock for system tuples is not preferable ?
> >
> > I think that UnlockRelation() is unnecessary fundamentally.
> > Mine is the simplest way to achieve this.
> > If there's no problem,I am glad to remove UnlockRelation() calls.
>
> There are! I finally found where I used UnlockRelation() -
> in execUtils.c:ExecCloseIndices(). Please read comments in
> ExecOpenIndices() where LockRelation() is called...
I see it now.
Hmm,index itself doesn't have its time qualification and is out of
transaction control(at least now).
OK,I would examine it one by one.
Regards.
Hiroshi Inoue
Inoue(at)tpf(dot)co(dot)jp
From | Date | Subject | |
---|---|---|---|
Next Message | FM&NB | 1999-12-01 13:47:07 | |
Previous Message | Vadim Mikheev | 1999-12-01 10:49:47 | Re: [HACKERS] Re: [GENERAL] drop/rename table and transactions |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 1999-12-01 16:15:04 | Re: [HACKERS] Re: [GENERAL] drop/rename table and transactions |
Previous Message | Zeugswetter Andreas SEV | 1999-12-01 10:56:15 | AW: [HACKERS] Insert into view |