Skip to content
This repository has been archived by the owner on Jun 7, 2021. It is now read-only.

[TRAFODION-2164] Incremental Update Statistics documentation #707

Merged
merged 2 commits into from Sep 19, 2016

Conversation

DaveBirdsall
Copy link
Contributor

@DaveBirdsall DaveBirdsall commented Sep 15, 2016

Added documentation for incremental UPDATE STATISTICS to the SQL Reference Manual.

Also deleted old change history info (that is, the change history for version 2.0 and 2.0.1).

Also updated the revision text (which goes in all manuals) to have a dummy entry for Trafodion version 2.1.0. A date will need to be supplied when this is released.

Added error messages for incremental UPDATE STATISTICS to the Messages Manual. While I was at it, reworked the error messages for UPDATE STATISTICS in general as it was incomplete and out-of-date. I moved these messages out of the "Sort Messages" chapter (they didn't belong there) to a separate, new chapter, "Update Statistics Messages".

@Traf-Jenkins
Copy link

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/1143/

@Traf-Jenkins
Copy link

Copy link
Contributor

@svarnau svarnau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only comment is on revisions section. We can keep it if updating a single file is reflected in all the docs.

@@ -26,6 +26,7 @@
[cols="2",options="header"]
|===
| Version | Date
| 2.1.0 | TBD
| 2.0.1 | July 7, 2016
| 2.0.0 | June 6, 2016
| 1.3.0 | January, 2016
Copy link
Contributor

@svarnau svarnau Sep 15, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess my preference was to remove this section, so we don't try to track release dates in documents.

Though it looks like this file is shared by all the docs? That is less bad than I thought. We could keep it if updating it is added to https://cwiki.apache.org/confluence/display/TRAFODION/Create+Release#CreateRelease-UpdateProjectWebSite

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it is shared in all documents. My thought was that perhaps the release manager could supply the date at the tail end of the release cycle. If this is agreeable, I can add a comment to the wiki as you suggest.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that sounds good. Just add revisions.txt to the list of those files that needs to be updated and docs published. Thanks.

@DaveBirdsall
Copy link
Contributor Author

Please do not merge this just yet. I forgot to add the error messages to the Messages Manual. Will submit a rework shortly.

@Traf-Jenkins
Copy link

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/1146/

@Traf-Jenkins
Copy link

@asfgit asfgit merged commit a4a538e into apache:master Sep 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants