Ticket #113 (closed enhancement: invalid)

Opened 16 months ago

Last modified 10 months ago

postgresql storage should not be using serializable transactions

Reported by: jack Owned by: smoku
Priority: major Component: storage
Version: 2.2 Keywords:
Cc: Tracforge_linkmap:
Blocking: Blocked By:

Description

This is not banking software. I believe that the original author must have used serializable because he was unfamiliar with transaction levels. We should be using the normal read committed level, which is enough for the purpose at hand. After all, there is no retry code if a serializable transaction fails is there? So it is implemented incorrectly anyway.

Also, we need to get rid of the overhead of setting the transaction isolation level for every transaction. This can be done when the connection is established and will save a lot of time.

Also, we should not be using BEGIN/END blocks on singleton inserts.

We're working on paches for all these, but I wanted to open a bug here for feedback and to track the status.

Change History

Changed 16 months ago by smoku

  • version changed from 2.1.7 to 2.2
  • type changed from defect to enhancement

Changed 16 months ago by smoku

I'm very open to the patches.

Changed 10 months ago by smoku

  • status changed from new to closed
  • resolution set to invalid

There is no reason for patches really.

Just comment-out <transactions/> in sm.xml and you're done.

Note: See TracTickets for help on using tickets.