2016-04-15 1 views
0

Keys定数を2回参照すると、jOOq 3.7.3にバグがあるようです。jOOqキーが不正クエリを生成する

select 
    "SCHEMA_A"."LIBRARY_ITEMS"."LIBRARY_ITEM_ID", 
    "SCHEMA_A"."LIBRARY_ITEMS"."TITLE", 
    "SCHEMA_A"."LIBRARY_ITEMS"."DETAILED_DESCRIPTION", 
    "SCHEMA_A"."LIBRARY_ITEMS"."URL", 
    "SCHEMA_A"."LIBRARY_ITEMS"."CREATE_DATE", 
    "SCHEMA_A"."LIBRARY_ITEMS"."LIBRARY_ID", 
    "SCHEMA_A"."LIBRARY_ITEMS"."ORGANIZATION_ID", 
    case when "AuxLeadCenter"."ORGANIZATION_ID" is not null then "AuxLeadCenter"."ORGANIZATION_NAME" 
     else "LeadCenter"."ORGANIZATION_NAME" 
    end "ORG_NAME", 
    case when "AuxLeadCenter"."ORGANIZATION_ID" is not null then "AuxLeadCenter"."ACRONYM" 
     else "LeadCenter"."ACRONYM" 
    end "ORG_ACRONYM", 
    "RespMissionDir"."ORGANIZATION_NAME" 
from "SCHEMA_A"."LIBRARY_ITEMS" 
    join (
    "SCHEMA_A"."LKU_CODE" 
     join "SCHEMA_A"."LKU_CODE_TYPE" 
     on (
     "SCHEMA_A"."LKU_CODE"."LKU_CODE_TYPE_ID" = "SCHEMA_A"."LKU_CODE_TYPE"."LKU_CODE_TYPE_ID" 
     and "SCHEMA_A"."LKU_CODE_TYPE"."CODE_TYPE" = 'LIBRARY_ITEM_TYPE' 
    ) 
) 
    on (
    "SCHEMA_A"."LIBRARY_ITEMS"."LIBRARY_ITEM_TYPE_ID" = "SCHEMA_A"."LKU_CODE"."LKU_CODE_ID" 
    and "SCHEMA_A"."LKU_CODE"."CODE" = 'NEWS_STORY' 
) 
    join (
    "SCHEMA_A"."LIBRARY" 
     join "SCHEMA_A"."LKU_CODE" 
     on (
     "SCHEMA_A"."LIBRARY"."ACTIVITY_TYPE_ID" = "SCHEMA_A"."LKU_CODE"."LKU_CODE_ID" 
     and "SCHEMA_A"."LKU_CODE"."CODE" in (
      'MISSION_WITH_TECHNOLOGY', 'TECHNOLOGY_ONLY' 
     ) 
    ) 
     join "SCHEMA_A"."LKU_CODE_TYPE" 
     on (
     "SCHEMA_A"."LKU_CODE"."LKU_CODE_TYPE_ID" = "SCHEMA_A"."LKU_CODE_TYPE"."LKU_CODE_TYPE_ID" 
     and "SCHEMA_A"."LKU_CODE_TYPE"."CODE_TYPE" = 'ACTIVITY_TYPE' 
    ) 
) 
    on "SCHEMA_A"."LIBRARY_ITEMS"."LIBRARY_ID" = "SCHEMA_A"."LIBRARY"."LIBRARY_ID" 
    join (
    "SCHEMA_A"."AC" 
     join (
     "SCHEMA_A"."LKU_CODE" 
      join "SCHEMA_A"."LKU_CODE_TYPE" 
      on (
      "SCHEMA_A"."LKU_CODE"."LKU_CODE_TYPE_ID" = "SCHEMA_A"."LKU_CODE_TYPE"."LKU_CODE_TYPE_ID" 
      and "SCHEMA_A"."LKU_CODE_TYPE"."CODE_TYPE" = 'OBJECT_TYPE' 
     ) 
    ) 
     on (
     "SCHEMA_A"."AC"."OBJECT_TYPE_ID" = "SCHEMA_A"."LKU_CODE"."LKU_CODE_ID" 
     and "SCHEMA_A"."LKU_CODE"."CODE" = 'PORTFOLIOS' 
     and exists (
      select 
      "SCHEMA_A"."AC_RELEASE_TYPES"."AC_ID", 
      "SCHEMA_A"."AC_RELEASE_TYPES"."SECTION", 
      "SCHEMA_A"."AC_RELEASE_TYPES"."RELEASE_TYPE", 
      "SCHEMA_A"."AC_RELEASE_TYPES"."CREATE_DATE", 
      "SCHEMA_A"."AC_RELEASE_TYPES"."CREATED_BY", 
      "SCHEMA_A"."AC_RELEASE_TYPES"."UPDATE_DATE", 
      "SCHEMA_A"."AC_RELEASE_TYPES"."UPDATED_BY" 
      from "SCHEMA_A"."AC_RELEASE_TYPES" 
      where (
      "SCHEMA_A"."AC"."AC_ID" = "SCHEMA_A"."AC_RELEASE_TYPES"."AC_ID" 
      and "SCHEMA_A"."AC_RELEASE_TYPES"."SECTION" = 1 
      and "SCHEMA_A"."AC_RELEASE_TYPES"."RELEASE_TYPE" in (
       6, 7 
      ) 
     ) 
     ) 
    ) 
     join "SCHEMA_A"."TP_USERS" 
     on (
     "SCHEMA_A"."TP_USERS"."USER_ID" = 456920 
     and exists (
      select 
      "SCHEMA_A"."AC_SENSITIVITIES"."AC_ID", 
      "SCHEMA_A"."AC_SENSITIVITIES"."SECTION", 
      "SCHEMA_A"."AC_SENSITIVITIES"."SENSITIVITIES", 
      "SCHEMA_A"."AC_SENSITIVITIES"."CREATE_DATE", 
      "SCHEMA_A"."AC_SENSITIVITIES"."CREATED_BY", 
      "SCHEMA_A"."AC_SENSITIVITIES"."UPDATE_DATE", 
      "SCHEMA_A"."AC_SENSITIVITIES"."UPDATED_BY" 
      from "SCHEMA_A"."AC_SENSITIVITIES" 
      where (
      "SCHEMA_A"."AC"."AC_ID" = "SCHEMA_A"."AC_SENSITIVITIES"."AC_ID" 
      and "SCHEMA_A"."AC_SENSITIVITIES"."SECTION" = 0 
      and bitand(
       "SCHEMA_A"."TP_USERS"."SENSITIVITIES", 
       "SCHEMA_A"."AC_SENSITIVITIES"."SENSITIVITIES") = "SCHEMA_A"."AC_SENSITIVITIES"."SENSITIVITIES" 
     ) 
     ) 
    ) 
) 
    on "SCHEMA_A"."LIBRARY"."LIBRARY_ID" = "SCHEMA_A"."AC"."OBJECT_ID" 
    join (
    "SCHEMA_A"."AC" 
     join (
     "SCHEMA_A"."LKU_CODE" 
      join "SCHEMA_A"."LKU_CODE_TYPE" 
      on (
      "SCHEMA_A"."LKU_CODE"."LKU_CODE_TYPE_ID" = "SCHEMA_A"."LKU_CODE_TYPE"."LKU_CODE_TYPE_ID" 
      and "SCHEMA_A"."LKU_CODE_TYPE"."CODE_TYPE" = 'OBJECT_TYPE' 
     ) 
    ) 
     on (
     "SCHEMA_A"."AC"."OBJECT_TYPE_ID" = "SCHEMA_A"."LKU_CODE"."LKU_CODE_ID" 
     and "SCHEMA_A"."LKU_CODE"."CODE" = 'LIBRARY_ITEMS' 
     and exists (
      select 
      "SCHEMA_A"."AC_SENSITIVITIES"."AC_ID", 
      "SCHEMA_A"."AC_SENSITIVITIES"."SECTION", 
      "SCHEMA_A"."AC_SENSITIVITIES"."SENSITIVITIES", 
      "SCHEMA_A"."AC_SENSITIVITIES"."CREATE_DATE", 
      "SCHEMA_A"."AC_SENSITIVITIES"."CREATED_BY", 
      "SCHEMA_A"."AC_SENSITIVITIES"."UPDATE_DATE", 
      "SCHEMA_A"."AC_SENSITIVITIES"."UPDATED_BY" 
      from "SCHEMA_A"."AC_SENSITIVITIES" 
      where (
      "SCHEMA_A"."AC_SENSITIVITIES"."AC_ID" = "SCHEMA_A"."AC"."AC_ID" 
      and "SCHEMA_A"."AC_SENSITIVITIES"."SECTION" = 0 
      and "SCHEMA_A"."AC_SENSITIVITIES"."SENSITIVITIES" = 0 
     ) 
     ) 
    ) 
) 
    on "SCHEMA_A"."LIBRARY_ITEMS"."LIBRARY_ITEM_ID" = "SCHEMA_A"."AC"."OBJECT_ID" 
    left outer join (
    "SCHEMA_A"."LIBRARY_ORGANIZATIONS" 
     join "SCHEMA_A"."ORGANIZATIONS" "RespMissionDir" 
     on "SCHEMA_A"."LIBRARY_ORGANIZATIONS"."ORGANIZATION_ID" = "RespMissionDir"."ORGANIZATION_ID" 
     join "SCHEMA_A"."LKU_CODE" 
     on (
     "SCHEMA_A"."LIBRARY_ORGANIZATIONS"."ASSOCIATION_TYPE_ID" = "SCHEMA_A"."LKU_CODE"."LKU_CODE_ID" 
     and "SCHEMA_A"."LKU_CODE"."CODE" = 'RESPONSIBLE_MISSION_DIRECTORATE' 
    ) 
     join "SCHEMA_A"."LKU_CODE_TYPE" 
     on (
     "SCHEMA_A"."LKU_CODE"."LKU_CODE_TYPE_ID" = "SCHEMA_A"."LKU_CODE_TYPE"."LKU_CODE_TYPE_ID" 
     and "SCHEMA_A"."LKU_CODE_TYPE"."CODE_TYPE" = 'PORTFOLIO_ORGANIZATION_ASSOCIATION_TYPE' 
    ) 
) 
    on "SCHEMA_A"."LIBRARY_ORGANIZATIONS"."LIBRARY_ID" = "SCHEMA_A"."LIBRARY"."LIBRARY_ID" 
    left outer join (
    "SCHEMA_A"."LIBRARY_ORGANIZATIONS" 
     join (
     "SCHEMA_A"."ORGANIZATIONS" "LeadCenter" 
      left outer join "SCHEMA_A"."ORGANIZATIONS" "AuxLeadCenter" 
      on (
      "LeadCenter"."REPLACEMENT_ORGANIZATION_ID" = "AuxLeadCenter"."ORGANIZATION_ID" 
      and "LeadCenter"."IS_ACTIVE" = 0 
     ) 
    ) 
     on "SCHEMA_A"."LIBRARY_ORGANIZATIONS"."ORGANIZATION_ID" = "LeadCenter"."ORGANIZATION_ID" 
     join "SCHEMA_A"."LKU_CODE" 
     on (
     "SCHEMA_A"."LIBRARY_ORGANIZATIONS"."ASSOCIATION_TYPE_ID" = "SCHEMA_A"."LKU_CODE"."LKU_CODE_ID" 
     and "SCHEMA_A"."LKU_CODE"."CODE" = 'LEAD_CENTER' 
    ) 
     join "SCHEMA_A"."LKU_CODE_TYPE" 
     on (
     "SCHEMA_A"."LKU_CODE"."LKU_CODE_TYPE_ID" = "SCHEMA_A"."LKU_CODE_TYPE"."LKU_CODE_TYPE_ID" 
     and "SCHEMA_A"."LKU_CODE_TYPE"."CODE_TYPE" = 'PORTFOLIO_ORGANIZATION_ASSOCIATION_TYPE' 
    ) 
) 
    on "SCHEMA_A"."LIBRARY_ORGANIZATIONS"."LIBRARY_ID" = "SCHEMA_A"."LIBRARY_ORGANIZATIONS"."LIBRARY_ID" 
order by case when "SCHEMA_A"."LIBRARY_ITEMS"."UPDATE_DATE" is null then "SCHEMA_A"."LIBRARY_ITEMS"."CREATE_DATE" 
       else "SCHEMA_A"."LIBRARY_ITEMS"."UPDATE_DATE" 
     end desc 

私はKeys.PO_P ___ FKに2つ目の参照を削除し、実際のキーと交換することが分かっ回避策:(のOracle 12C)悪いSQLクエリを生成

final Select<Record10<BigDecimal, String, String, String, Timestamp, BigDecimal, BigDecimal, String, String, String>> query = readContext 
      .select(li.LIBRARY_ITEM_ID, 
        li.TITLE, 
        li.DETAILED_DESCRIPTION, 
        li.URL, 
        li.CREATE_DATE, 
        li.PORTFOLIO_ID, 
        li.ORGANIZATION_ID, 
        DSL.decode() 
          .when(o_aux_lead_center.ORGANIZATION_ID.isNotNull(), 
            o_aux_lead_center.ORGANIZATION_NAME) 
          .otherwise(o_lead_center.ORGANIZATION_NAME).as(ORG_NAME), 
        DSL.decode() 
          .when(o_aux_lead_center.ORGANIZATION_ID.isNotNull(), 
            o_aux_lead_center.ACRONYM).otherwise(o_lead_center.ACRONYM) 
          .as(ORG_ACRONYM), o_resp_mission_dir.ORGANIZATION_NAME) 
      .from(li 
        .join(lc.join(lct).onKey(Keys.LC_LCT___FK) 
          .and(lct.CODE_TYPE.equal(LkuCodeTypeLookup.LIBRARY_ITEM_TYPE))) 
        .onKey(Keys.LI_LC_LITI___FK) 
        .and(lc.CODE.equal(LkuCodeLookup.LIBRARY_ITEM_TYPE_NEWS_STORY)) 
        // The Portfolio 
        .join(p.join(lc).onKey(Keys.P_LC_ATI___FK) 
          .and(lc.CODE.in(LkuCodeLookup.PORTFOLIO_ACTIVITY_TYPE_TECHNOLOGY)) 
          .join(lct).onKey(Keys.LC_LCT___FK) 
          .and(lct.CODE_TYPE.equal(LkuCodeTypeLookup.ACTIVITY_TYPE))) 
        .on(li.PORTFOLIO_ID.equal(p.PORTFOLIO_ID)) 
        // The CO for the Portfolio associated with the library item 
        .join(co 
          .join(lc.join(lct).onKey(Keys.LC_LCT___FK) 
            .and(lct.CODE_TYPE.equal(LkuCodeTypeLookup.OBJECT_TYPE))) 
          .onKey(Keys.CO_LC_OTI___FK) 
          .and(lc.CODE.equal(LkuCodeLookup.OBJECTTYPE_PORTFOLIOS)) 
          .and(limitByRelease).join(tu).on(tu.USER_ID.equal(uid)) 
          .and(limitBySensitivities)) 
        .on(p.PORTFOLIO_ID.equal(co.OBJECT_ID)) 
        // The CO for the Library Item 
        .join(co 
          .join(lc.join(lct).onKey(Keys.LC_LCT___FK) 
            .and(lct.CODE_TYPE.equal(LkuCodeTypeLookup.OBJECT_TYPE))) 
          .onKey(Keys.CO_LC_OTI___FK) 
          .and(lc.CODE.equal(LkuCodeLookup.OBJECTTYPE_LIBRARY_ITEMS)) 
          .andExists(
            readContext 
              .selectFrom(cos) 
              .where(cos.CORE_OBJECT_ID.equal(co.CORE_OBJECT_ID)) 
              .and(cos.SECTION 
                .equal(BigDecimal 
                  .valueOf(AccessControlConstants.CORE_OBJECT_CHAR_ARRAY_SELF)) 
                .and(cos.SENSITIVITIES.equal(BigDecimal.ZERO))))) 
        .on(li.LIBRARY_ITEM_ID.equal(co.OBJECT_ID))) 
      // The Responsible Mission Directorate 
      .leftOuterJoin(
        po 
          .join(o_resp_mission_dir) 
          .on(po.ORGANIZATION_ID.equal(o_resp_mission_dir.ORGANIZATION_ID)) 
          .join(lc) 
          .onKey(Keys.PO_LC_ATI___FK) 
          .and(lc.CODE 
            .equal(LkuCodeLookup.ASSOCIATIONTYPE_RESPONSIBLE_MISSION_DIRECTORATE)) 
          .join(lct) 
          .onKey(Keys.LC_LCT___FK) 
          .and(lct.CODE_TYPE 
            .equal(LkuCodeTypeLookup.PORTFOLIO_ORGANIZATION_ASSOCIATION_TYPE))) 
      .onKey(Keys.PO_P___FK) 
      // The Portfolio Organizations 
      .leftOuterJoin(
        po 
          // Add in the organization data 
          .join(o_lead_center.leftOuterJoin(o_aux_lead_center).on(
            o_lead_center.REPLACEMENT_ORGANIZATION_ID.equal(
              o_aux_lead_center.ORGANIZATION_ID).and(
              o_lead_center.IS_ACTIVE.equal(byteZero)))) 
          .on(po.ORGANIZATION_ID.equal(o_lead_center.ORGANIZATION_ID)) 
          .join(lc) 
          .onKey(Keys.PO_LC_ATI___FK) 
          .and(lc.CODE.equal(LkuCodeLookup.ASSOCIATIONTYPE_LEAD_CENTER)) 
          .join(lct) 
          .onKey(Keys.LC_LCT___FK) 
          .and(lct.CODE_TYPE 
            .equal(LkuCodeTypeLookup.PORTFOLIO_ORGANIZATION_ASSOCIATION_TYPE))) 
      .onKey(Keys.PO_P___FK) 
      .orderBy(
        DSL.decode().when(li.UPDATE_DATE.isNull(), li.CREATE_DATE) 
          .otherwise(li.UPDATE_DATE).desc()); 

:以下の例を見てみましょうしかし、これを見ている人は誰ですか?誰もそれが正しく"SCHEMA_A"."LIBRARY_ORGANIZATIONS"."LIBRARY_ID" = "SCHEMA_A"."LIBRARY"."LIBRARY_ID"にデータを設定する理由を知っていますか?しかし、それは"SCHEMA_A"."LIBRARY_ORGANIZATIONS"."LIBRARY_ID" = "SCHEMA_A"."LIBRARY_ORGANIZATIONS"."LIBRARY_ID"にオンに設定されているため、2番目にクロス結合が起こりますか?

+0

ここでの問題は、2番目の 'onKey()'メソッドが呼び出された段階で、結合テーブルツリーで 'LIBRARY_ID'カラムがあいまいであることでしょう。ログに警告が表示されますか?あいまいな検索が行われたときの、jOOQ 3.7.3以降の警告があるはずです(https://github.com/jOOQ/jOOQ/issues/5047) –

+0

私はあなたが話している警告を見ました。私が 'onKey(ForeignKey key)'呼び出しをしたときではなく、サブジョインを持つものに加わっている場合は空の 'onKey()'コールを返します。私は実際にjOOq(foreign key key) 'を呼び出しましたが、エイリアステーブル(jooq)は許可されていますが、SQL例外が発生しました。私が尋ねたが、エイリアスを使用しなかったのと同じように、ForeignKeyで。好奇心のために、エイリアステーブルからFKを引き出す良い方法はありますか? – DanO

+0

修正されるべき問題がありました:https://github.com/jOOQ/jOOQ/issues/2870。問題が依然として続く場合、または修正が完了していない場合は、別の質問の価値がありますか? –

答えて

1

ON KEY結合方法は、通常のON結合述語に変換されます。これは、両方の "最良の"一致するテーブルを見つけることによって行われます。キーの外部キーテーブルがの最初のに一致するバグ(#5209)があり、一致すると主キーテーブルは検証されません。このため、キーは間違った方向に解決されます。クエリで

は、"SCHEMA_A"."LIBRARY_ORGANIZATIONS"テーブルをリネームすることなく、二回参加している:

left outer join (
    "SCHEMA_A"."LIBRARY_ORGANIZATIONS" 
     join "SCHEMA_A"."ORGANIZATIONS" "RespMissionDir" 

そして:

left outer join (
    "SCHEMA_A"."LIBRARY_ORGANIZATIONS" 
     join (
     "SCHEMA_A"."ORGANIZATIONS" "LeadCenter" 

この曖昧さはjOOQによって正しく解決することはできません。 onKey()の使用があいまいでは解決できないことを示す警告が、jOOQ 3.7.3(#5047)のログファイルに表示されます。

一般に、結合テーブルツリーが複雑すぎる場合はonKey()を避けることをお勧めします。

+0

それを爆破する(あなたの最後の文に関して)!あなたは 'onKey(Keys.XXX)'の呼び出しをどれくらい気に入っていますか?実際には、私は本当に複数のFKを汲み取ることができたいと思っています。私は 'onKey(Key.%ChildTableAcr%_%ParentTableAcr%___ FK)'を実行するだけでFKが本当に何を言っているのか心配する必要はないことが大好きです(変更すればjOOqコードを再構築するだけです) 。 体は本当に曖昧ですか? 'onKey'を' Keys'エントリで呼びます。 – DanO

+0

'onKey()'を使うのはいいですが、明示的ではないので大変危険です。 。クエリが大きくなるにつれて、セマンティクスは最初は不明確になる可能性があります。私はあなたがこの場合正しいと思うし、バグがある。あいまいさは、キーの方向が外部キーテーブル(曖昧)に基づいてのみ検出され、主キーテーブル(曖昧ではない)に基づいて検出されないためにのみ発生します。私はこれのために問題を作りました:https://github.com/jOOQ/jOOQ/issues/5209。報告ありがとう! –

+0

あなたは簡単な再生手段を見つけたことをうれしく思っています!私はそれがそれらの複雑なものの一つかもしれないことを恐れていました。チケットを開けて、説明/討論をしてくれてありがとう! – DanO

関連する問題