記一個BUG的解決過程

問題

Gem版本:
rails 4.2.5
activerecord-oracle_enhanced-adapter 1.6.6

兩個模型:

class ProjectTable < ActiveRecord::Base

    establish_connection :projects
        ...
end

class TrialApplication < ProjectTable

  self.table_name =  'trial_applications'
  
end

BUG:
TrialApplication.create的時候,ID本來應該是取自序列trial_application_seq.nextval,但是發現完全不對,創建數據的ID與已有的ID發生沖突,引發ORA-00001報錯!

追蹤

初步懷疑

因為在Rails中,Oracle和Mysql有點不同,Mysql的主鍵ID是自增長的,而Oracle一般都是通過序列來獲取,一開始就懷疑新版的activerecord-oracle_enhanced-adapter 是否有調整,造成通過序列獲取ID出現了問題。于是開始追蹤activerecord-oracle_enhanced-adapter 的源代碼。

開始追蹤

于是我通過 sequence 這個關鍵詞在activerecord-oracle_enhanced-adapter 的源代碼中進行搜索,查到了這個方法(activerecord-oracle_enhanced-adapter-1.6.6/lib/active_record/connection_adapters/oracle_enhanced/database_statements.rb):

    # Executes an INSERT statement and returns the new record's ID
    def insert_sql(sql, name = nil, pk = nil, id_value = nil, sequence_name = nil) #:nodoc:
      # if primary key value is already prefetched from sequence
      # or if there is no primary key
      if id_value || pk.nil?
        execute(sql, name)
        return id_value
      end

      sql_with_returning = sql + @connection.returning_clause(quote_column_name(pk))
      log(sql, name) do
        @connection.exec_with_returning(sql_with_returning)
      end
    end        
    
    # New method in ActiveRecord 3.1
    # Will add RETURNING clause in case of trigger generated primary keys
    def sql_for_insert(sql, pk, id_value, sequence_name, binds)
      unless id_value || pk.nil? || (defined?(CompositePrimaryKeys) && pk.kind_of?(CompositePrimaryKeys::CompositeKeys))
        sql = "#{sql} RETURNING #{quote_column_name(pk)} INTO :returning_id"
        returning_id_col = new_column("returning_id", nil, Type::Value.new, "number", true, "dual", true, true)
        (binds = binds.dup) << [returning_id_col, nil]
      end
      [sql, binds]
    end

這兩個方法很明顯就是生成insert的sql的方法,應該就是倒推過來最接近數據庫操作的步驟,于是我在兩個方法中加上了輸出:

puts [sql, id_value, sequence_name]

然后返回控制臺中執行TrialApplication.create操作,結果返回的是:

["INSERT INTO \"TRIAL_APPLICATIONS\" (...)") VALUES (:a1, :a2, :a3, :a4, :a5, :a6, :a7)", 123456, nil]

很明顯,這個時候id_value早就已經取好值了,很明顯,還要繼續追溯這個id_value是在哪取出來的。

繼續深挖

activerecord-oracle_enhanced-adapter 的源代碼挖了一遍,沒有收獲,于是開始搜索active_record,還是用sequence關鍵詞,發現這個方法(activerecord-4.2.5/lib/active_record/connection_adapters/abstract/database_statements.rb ):

  # Returns the last auto-generated ID from the affected table.
  #
  # +id_value+ will be returned unless the value is nil, in
  # which case the database will attempt to calculate the last inserted
  # id and return that value.
  #
  # If the next id was calculated in advance (as in Oracle), it should be
  # passed in as +id_value+.
  def insert(arel, name = nil, pk = nil, id_value = nil, sequence_name = nil, binds = [])
    puts 'insert'   #這個就是我加入的調試的代碼了
    puts [name, pk, id_value, sequence_name, binds].inspect
    sql, binds = sql_for_insert(to_sql(arel, binds), pk, id_value, sequence_name, binds)
    value      = exec_insert(sql, name, binds, pk, sequence_name)
    id_value || last_inserted_id(value)
  end

注釋里面寫的很清楚了,If the next id was calculated in advance (as in Oracle),在Oracle中,id是會預先計算出來的了,加上調試的代碼,驗證,果然在這一步id_value也早就計算出來,還是錯誤的!

最后的希望

繼續檢索,終于找到了這個方法(activerecord-4.2.5/lib/active_record/relation.rb),

  def insert(values) # :nodoc:
    primary_key_value = nil

    if primary_key && Hash === values
      primary_key_value = values[values.keys.find { |k|
        k.name == primary_key
      }]

      if !primary_key_value && connection.prefetch_primary_key?(klass.table_name)
        puts klass.sequence_name
        primary_key_value = connection.next_sequence_value(klass.sequence_name)
        puts "primary_key_value=#{primary_key_value}"
        values[klass.arel_table[klass.primary_key]] = primary_key_value
      end
    end
    ....

connection.next_sequence_value(klass.sequence_name),獲取序列的下一個值的方法就是它了,說明id_value就是在這里取出來的。

加上調試的輸出,結果一看:
klass.sequence_name居然不是trial_applications_seq,而是employees_seq!問題終于找到了!

解決

原來在ProjectTable中,新版的activerecord會預先加載模型對應的table的信息,避免報錯我加上了self.table_name = 'employees',指定了一個table。

而TrialApplication,就悲催的延續了ProjectTable的設置,雖然self.table_name = 'trial_applications'修改了表名,序列sequence_name沒有修改!解決起來就很簡單了:

class TrialApplication < ProjectTable

  self.table_name =  'trial_applications'
  self.sequence_name = 'trial_applications_seq'
end

加上 self.sequence_name = 'trial_applications_seq'搞定!

最后編輯于
?著作權歸作者所有,轉載或內容合作請聯系作者
平臺聲明:文章內容(如有圖片或視頻亦包括在內)由作者上傳并發布,文章內容僅代表作者本人觀點,簡書系信息發布平臺,僅提供信息存儲服務。
  • 序言:七十年代末,一起剝皮案震驚了整個濱河市,隨后出現的幾起案子,更是在濱河造成了極大的恐慌,老刑警劉巖,帶你破解...
    沈念sama閱讀 229,885評論 6 541
  • 序言:濱河連續發生了三起死亡事件,死亡現場離奇詭異,居然都是意外死亡,警方通過查閱死者的電腦和手機,發現死者居然都...
    沈念sama閱讀 99,312評論 3 429
  • 文/潘曉璐 我一進店門,熙熙樓的掌柜王于貴愁眉苦臉地迎上來,“玉大人,你說我怎么就攤上這事。” “怎么了?”我有些...
    開封第一講書人閱讀 177,993評論 0 383
  • 文/不壞的土叔 我叫張陵,是天一觀的道長。 經常有香客問我,道長,這世上最難降的妖魔是什么? 我笑而不...
    開封第一講書人閱讀 63,667評論 1 317
  • 正文 為了忘掉前任,我火速辦了婚禮,結果婚禮上,老公的妹妹穿的比我還像新娘。我一直安慰自己,他們只是感情好,可當我...
    茶點故事閱讀 72,410評論 6 411
  • 文/花漫 我一把揭開白布。 她就那樣靜靜地躺著,像睡著了一般。 火紅的嫁衣襯著肌膚如雪。 梳的紋絲不亂的頭發上,一...
    開封第一講書人閱讀 55,778評論 1 328
  • 那天,我揣著相機與錄音,去河邊找鬼。 笑死,一個胖子當著我的面吹牛,可吹牛的內容都是我干的。 我是一名探鬼主播,決...
    沈念sama閱讀 43,775評論 3 446
  • 文/蒼蘭香墨 我猛地睜開眼,長吁一口氣:“原來是場噩夢啊……” “哼!你這毒婦竟也來了?” 一聲冷哼從身側響起,我...
    開封第一講書人閱讀 42,955評論 0 289
  • 序言:老撾萬榮一對情侶失蹤,失蹤者是張志新(化名)和其女友劉穎,沒想到半個月后,有當地人在樹林里發現了一具尸體,經...
    沈念sama閱讀 49,521評論 1 335
  • 正文 獨居荒郊野嶺守林人離奇死亡,尸身上長有42處帶血的膿包…… 初始之章·張勛 以下內容為張勛視角 年9月15日...
    茶點故事閱讀 41,266評論 3 358
  • 正文 我和宋清朗相戀三年,在試婚紗的時候發現自己被綠了。 大學時的朋友給我發了我未婚夫和他白月光在一起吃飯的照片。...
    茶點故事閱讀 43,468評論 1 374
  • 序言:一個原本活蹦亂跳的男人離奇死亡,死狀恐怖,靈堂內的尸體忽然破棺而出,到底是詐尸還是另有隱情,我是刑警寧澤,帶...
    沈念sama閱讀 38,998評論 5 363
  • 正文 年R本政府宣布,位于F島的核電站,受9級特大地震影響,放射性物質發生泄漏。R本人自食惡果不足惜,卻給世界環境...
    茶點故事閱讀 44,696評論 3 348
  • 文/蒙蒙 一、第九天 我趴在偏房一處隱蔽的房頂上張望。 院中可真熱鬧,春花似錦、人聲如沸。這莊子的主人今日做“春日...
    開封第一講書人閱讀 35,095評論 0 28
  • 文/蒼蘭香墨 我抬頭看了看天上的太陽。三九已至,卻和暖如春,著一層夾襖步出監牢的瞬間,已是汗流浹背。 一陣腳步聲響...
    開封第一講書人閱讀 36,385評論 1 294
  • 我被黑心中介騙來泰國打工, 沒想到剛下飛機就差點兒被人妖公主榨干…… 1. 我叫王不留,地道東北人。 一個月前我還...
    沈念sama閱讀 52,193評論 3 398
  • 正文 我出身青樓,卻偏偏與公主長得像,于是被迫代替她去往敵國和親。 傳聞我的和親對象是個殘疾皇子,可洞房花燭夜當晚...
    茶點故事閱讀 48,431評論 2 378

推薦閱讀更多精彩內容