stardog-union / rocksdb

GNU General Public License v2.0
4 stars 1 forks source link

Populate cf_id member of CompactionJobInfo for OnCompactionBegin #8

Closed matthewvon closed 4 years ago

matthewvon commented 4 years ago

Looks like somebody simply missed initializing a member variable. The column family ID, cf_id, is not set during OnCompactionBegin. But it is set properly in the next function for OnCompactionCompleted. Need this cf_id for tracking progress of a Stardog optimize since there may be multiple compactions required for a given column family.

I have put in a query to Facebook. Anticipate filing a PR there too since the line is still missing in the most current code.