--- David Crawshaw <[EMAIL PROTECTED]> wrote:
> Joe wrote:
> > It just occurred to me that a correctly working fsync() could
> > very well fix the file length caching problem without this patch.
> 
> I put up version 2007-01-07, which has a working fsync() in it.

The tar file nestedvm-2007-01-07.tgz contains the old no-op fsync() in
nestedvm-2007-01-07/src/org/ibex/nestedvm/support_aux.c. 

This led to some confusion when I tried to put a println() in the fsync 
implementation to prove it was being called on Windows and found it 
wasn't in Runtime.java. However, I do see your fsync fix in the tar 
file under:

_darcs/patches/20070107161020-0c629-f8cb28cb78260e588b7a1a5515d6a3a6c940a7f0.gz 

Once the fsync patch was manually applied, the sqlite test on Windows 
ran to completion without database corruption.

(offline if someone could tell me how to update my local src tree 
in darcs from a local repository, I'd appreciate it)

While the earlier HostFStat/size patch is not needed in this case,
it might be worth considering applying if only to make NestedVM
on Windows match Linux/UNIX's behavior in the absence of fsync.
I don't know what POSIX says about fstat() behavior in relation 
to not being fsync'd, but UNIXes don't seem to need it in this 
particular case.

Looking forward an ACID-compliant sqlite-3.3.10 version of your 
jdbc driver.

thanks.


 
____________________________________________________________________________________
Do you Yahoo!?
Everyone is raving about the all-new Yahoo! Mail beta.
http://new.mail.yahoo.com

--~--~---------~--~----~------------~-------~--~----~
 You received this message because you are subscribed to the Google Groups 
"SQLiteJDBC" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups-beta.google.com/group/sqlitejdbc?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to