You are viewing a plain text version of this content. The canonical link for it is here.
Posted to codereview@trafodion.apache.org by DaveBirdsall <gi...@git.apache.org> on 2018/01/28 05:02:53 UTC

[GitHub] trafodion pull request #1416: [TRAFODION-2929] Add *REBUILD INDEXES* for LOA...

Github user DaveBirdsall commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1416#discussion_r164288987
  
    --- Diff: docs/sql_reference/src/asciidoc/_chapters/sql_utilities.adoc ---
    @@ -517,6 +518,18 @@ Bulk Loader is executing.
     specifies that the target table, which is an index, be populated with
     data from the parent table.
     
    +** `REBUILD INDEXES`
    ++
    +specifies that indexes of the target table will be updated automatically when the source table 
    +is updated. 
    ++
    +This is the default behavior of the LOAD Statement, that is, even if this option is not 
    +specified, the LOAD Statement will rebuild indexes except the 
    +CQD `TRAF_LOAD_ALLOW_RISKY_INDEX_MAINTENANCE` is turned *ON*. This CQD is turned *OFF* by default, 
    --- End diff --
    
    "This CQD is turned *OFF* by default, ..." Suggest making this a complete sentence. What comes after it seems to stand well on its own.


---