DefaultDataSet is incompatible w/ AbstractDataSet's IDataSet

DefaultDataSet's implementation is incompatible w/ AbstractDataSet's implementation of getTable, getTables, and getTableName methods.

AbstractDataSet uses "private OrderedTableNameMap _orderedTableNameMap;" for table management.

DefaultDataSet uses "private final OrderedTableNameMap _tableMap;" for
table management.

That works fine so long as DefaultDataSet overrides every method of
AbstractDataSet, but it fails to override the getTable(String),
getTables(), and getTableNames() methods, which is half of the
IDataSet interface.

What this means is that if you construct a DefaultDataSet using the
DefaultDataSet methods, your constructed data set is stored in
_tableMap, but when you then refetch those values using the getters,
you're fetching out of the empty _orderedTableNameMap.

Ie, the following method fails because the previously-added tables are
not found, yet when we try to re-add the tables, we get an exception to that
effect.

public static DefaultTable getOrCreateTable(DefaultDataSet
aDataSet, String tableName) throws DataSetException,
FileNotFoundException, IOException
{
DefaultTable table = null;
try
{
table = (DefaultTable)aDataSet.getTable(tableName);
}
catch (NoSuchTableException e)
{
ITableMetaData userTableMetaData =
BaseDBunit.getMetaData().getTableMetaData(tableName);
table = new DefaultTable(userTableMetaData, new ArrayList());
aDataSet.addTable(table);
}

return table;
}

The solution would be to either override every method of
AbstractDataSet or to make _orderedTableNameMap protected and allow
subclasses to use it. Ie, replace _tableMap in DefaultDataSet with
_orderedTableNameMap.

I've created a patch to fix this that passes all 2.4.4 unit tests. I have not created a unit test that demonstrates the problem, however.

I have changed AbstractDataSet's _orderedTableNameMap. to protected.
I have replaced references with _tableMap with _orderedTableNameMap.
I have called initialize() at the appropriate places, and provided a dumbed-down implementation of initialize (since the one in AbstractDataSet does work that is unnecessary for DefaultDataSet).