lucenenet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Van Den Berghe, Vincent" <>
Subject RE: Pull requests
Date Tue, 13 Jun 2017 05:55:53 GMT
Hello Shad,

I've finally managed to do a pull request again. There must be something wrong with my brain.
I've pushed (2). I'll refrain to push (1).


-----Original Message-----
From: Shad Storhaug [] 
Sent: Monday, June 12, 2017 11:22 PM
Subject: RE: Pull requests



>From the documentation of MMapDirectory:

File-based Directory implementation that uses MemoryMappedFile for reading, and FSDirectory.FSIndexOutput
for writing.

I didn't see the point in maintaining the various write methods in the MemoryMappedFileByteBuffer,
since it is not used for writing anyway, so all of the Put() overloads just throw a NotSupportedException().


This sounds like a good fix to include.

Note that to make a pull request, you need to first make a branch on your end, push the branch
to GitHub, and then make the pull request (targeting master). This way you can have several
open pull requests at the same time that each pertain to something different.

Shad Storhaug (NightOwl888)

-----Original Message-----
From: Van Den Berghe, Vincent []
Sent: Monday, June 12, 2017 7:24 PM
Subject: Pull requests


I've done a couple of "pull requests",  but for some reason they do not appear. Maybe it's
just as well. They are based on "master", which I understand is the most recent version now.
I've made them on my current "clone" of the master. Maybe someone can review them and integrate
them as they please.

The two pull requests are:
This  implements the more efficient "Put" operation on memory mapped files, analogous to the
"Get" operations which was implemented by yours truly as well.

This implements "LUCENE-5644: switch to simpler LIFO thread to ThreadState allocator during
indexing". Technically, this is something from releases/lucene-solr/4.8.1, but profiling indicates
it makes a huge difference in multithreaded scenarios and I had to implement it.  Tests have
been adapted, and everything runs.


View raw message