Revision 3525aac9e5714be6635361dee5e320854ed814fc authored by Igor Canadi on 18 June 2014, 05:04:37 UTC, committed by Igor Canadi on 18 June 2014, 05:04:37 UTC
Summary:
This is minor, but if we put the writing talbe factory as the third parameter, when we add a new table format, we'll have a situation:
1) block based factory
2) plain table factory
3) output factory
4) new format factory

I think it makes more sense to have output as the first parameter.

Also, fixed a NewAdaptiveTableFactory() call in unit test

Test Plan: unit test

Reviewers: sdong

Reviewed By: sdong

Subscribers: leveldb

Differential Revision: https://reviews.facebook.net/D19119
1 parent 8c265c0
History

back to top