A never ending loop...not sure why?

Question

When I run my current procedure, it seems to be getting caught in a never ending loop, not entirley sure what the problem is...? I'm not quite the expert so if possible, the more detail in the explaniation the better...thanks!!
--The cursors
cursor konami_mlb_pitchers_crsr is
   select player_id_1032
   from customer_data.cd_baseball_pit_ytd_stats
   where split_number = -3
     and game_type_id = 1
     and sequence = 0
     and season_id = iSeasonID
   order by player_id_1032;
cursor konami_mlb_career_pit_crsr is
   select ytd.player_id_1032,
      ytd.player_id,
      ytd.moniker,
      ytd.last_name,
      sum(ytd.games_played),
      trunc(sum(ytd.outs_pitched) / 3) || '.' || mod(sum(ytd.outs_pitched), 3) as innings_pitched,
      sum(ytd.complete_games),
      sum(ytd.shut_outs),
      sum(ytd.wins),
      sum(ytd.losses),
      to_char((sum(ytd.wins)) / decode(sum(ytd.wins) + sum(ytd.losses), 0, 1, null, 1, sum(ytd.wins) + sum(ytd.losses)), '0d000') as winning_pct,
      trim(to_char( (9 * sum(ytd.earned_runs)) / (sum(ytd.outs_pitched) / 3) , '990d00')) as era,
      sum(ytd.strikeouts),
      sum(ytd.saves),
      sum(ytd.hits),
      sum(ytd.home_runs),
      sum(ytd.walks),
      sum(ytd.hit_batters),
      sum(ytd.earned_runs),
      sum(ytd.runs)
   from customer_data.cd_baseball_pit_ytd_stats ytd,  customer_data.cd_baseball_roster rost
   where ytd.split_number = -3
     and ytd.game_type_id = 1
     and ytd.sequence = 0
     and rost.player_id = ytd.player_id
     and rost.league_id = iLeagueID
     and ytd.season_id <= iSeasonId   
     and ytd.active_record != 'R'
     and ytd.player_id_1032 in
    --Includes Players that were on the DL for the entire season                                          
      select player_id_1032
     from customer_data.cd_baseball_roster r
     where r.year_last = 2008
     and r.league_id = iLeagueID
     and r.player_id_1032 is not NULL
     group by ytd.player_id_1032,ytd.player_id, ytd.moniker,  ytd.last_name;   --Where the cursor gets opened
open konami_mlb_pitchers_crsr;
   loop
      fetch konami_mlb_pitchers_crsr into pitcher;
      exit when konami_mlb_pitchers_crsr%notfound;
      open konami_mlb_career_pit_crsr;
      loop
         fetch konami_mlb_career_pit_crsr into pitcher_record;
         exit when konami_mlb_career_pit_crsr%notfound;
         utl_file.put_line(file_handle, pitcher_record.first_name || ',' ||
                                        pitcher_record.last_name || ',' ||
                                        pitcher_record.id || ',' ||
                                        pitcher_record.games_played || ',' ||
                                        pitcher_record.innings_pitched || ',' ||
                                        pitcher_record.complete_games || ',' ||
                                        pitcher_record.shutouts || ',' ||
                                        pitcher_record.wins || ',' ||
                                        pitcher_record.losses || ',' ||
                                        pitcher_record.winning_pct || ',' ||
                                        pitcher_record.era || ',' ||
                                        pitcher_record.strikeouts || ',' ||
                                        pitcher_record.saves || ',' ||
                                        pitcher_record.hits || ',' ||
                                        pitcher_record.homeruns || ',' ||
                                        pitcher_record.walks || ',' ||
                                        pitcher_record.hit_batters || ',' ||
                                        pitcher_record.earned_runs || ',' ||
                                        pitcher_record.runs);
      end loop; -- end loop through the stats for each pitcher
      close konami_mlb_career_pit_crsr;
   end loop; -- end loop through all the pitchers
   close konami_mlb_pitchers_crsr;

Answer 1

You don't seem to be using the cursor iterator variable "pitcher" when you print the data to the file. You are not even passing any parameter to the "konami_mlb_career_pit_crsr" cursor. As such the first cursor "konami_mlb_pitchers_crsr" appears to be redundant.
Maybe the loop is time-consuming (instead of "never-ending"), and if you wait long enough, it will end. In the meantime, you may want to tune your logic.
isotope