Está absolutamente correto. Cada recurso precisa de seu próprio `using` para garantir o encerramento dele quando não mais for necessário. Pode melhorar um pouco aqui:

    using (DataTable dataTable = new DataTable())
    using (MySqlDataAdapter mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand)) {
        mySqlDataAdapter.Fill(dataTable);
        return dataTable;
    }

É possível usar um bloco só com mais de um recursos aberto. Ao final do bloco, ambos serão encerrados.

Na verdade tem um problema aí. O código está retornando algo que está sendo disposto. Isto não vai dar certo. Se vai retornar o recurso para fora do método, você não pode dispor dele. Neste caso o `using` não deve ser usado para que o recurso permaneça vivo. Assim:

    DataTable dataTable = new DataTable();
    using (MySqlDataAdapter mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand)) {
        mySqlDataAdapter.Fill(dataTable);
        return dataTable;
    }

Ou em alguns casos (não nesse):

    DataTable dataTable = new DataTable();
    try {
        using (MySqlDataAdapter mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand)) {
            mySqlDataAdapter.Fill(dataTable);
            return dataTable;
        }
    } finally {
        if (dataTable != null) {
            dataTable.Dispose();
        }
    }