[GitHub] incubator-trafodion pull request #895: [TRAFODION-2412]update sql reference ...

2016-12-26 Thread liuyu000
Github user liuyu000 closed the pull request at:

https://github.com/apache/incubator-trafodion/pull/895


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafodion pull request #895: [TRAFODION-2412]update sql reference ...

2016-12-26 Thread traflm
Github user traflm commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/895#discussion_r93874161
  
--- Diff: 
docs/sql_reference/src/asciidoc/_chapters/sql_language_elements.adoc ---
@@ -137,6 +137,74 @@ You can define specific default settings for columns 
when the table is created.
 default settings for columns within tables. The default setting for a 
column is the value inserted in a row when an INSERT
 statement omits a value for a particular column.
 
+<<<
+[[identity_columns]]
+== Identity Columns
+
+Trafodion introduces the ability defining an identity clause against a 
table column defined using a numeric type. The syntax is shown below.
+
+```
+GENERATED
+[ALWAYS | BY DEFAULT]
+AS IDENTITY [(identity_options)]
+```
+
+Ignoring the identity_options, which match those of the 
<>, this syntax allows us 
to use three variations on the identity functionality.
+
+[[syntax_description_of_identity_columns]]
+=== Syntax Description of Identity Columns
+* `ALWAYS`
++
+Using ALWAYS forces the use of the identity. If an insert statement 
references the identity column, even to specify a NULL value, an error is 
produced.
+
+* `BY DEFAULT`
++
+Using BY DEFAULT allows you to use the identity if the column isn't 
referenced in the insert statement, but if the column is referenced, the 
specified value will be used in place of the identity. Attempting to specify 
the value NULL in this case results in an error, since identity columns are 
always NOT NULL.
--- End diff --

line is too long, need to limit each line's length, to generate a better 
HTML displaying. 
This is a general comment, please consider to change all 'long lines' below 
into fixed width line.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafodion pull request #896: [TRAFODION-2410] Support install Traf...

2016-12-26 Thread mkby
GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/896

[TRAFODION-2410] Support install Trafodion on customized CDH parcel d…

…irectory

also changes option name from 'upgrade' to 'reinstall'.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2410

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/896.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #896


commit 9a888d2034748d4f9a60fb1d621f2c010c36ebb4
Author: Eason 
Date:   2016-12-26T08:40:55Z

[TRAFODION-2410] Support install Trafodion on customized CDH parcel 
directory




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---