You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-dev@db.apache.org by "Kathey Marsden (JIRA)" <de...@db.apache.org> on 2005/12/01 01:54:30 UTC
[jira] Created: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
-----------------------------------------------------------------------------
Key: DERBY-737
URL: http://issues.apache.org/jira/browse/DERBY-737
Project: Derby
Type: Improvement
Components: SQL
Versions: 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.1.1, 10.1.1.2, 10.1.2.0, 10.1.2.1, 10.2.0.0, 10.1.3.0, 10.1.2.2, 10.0.2.2
Reporter: Kathey Marsden
Priority: Minor
There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
The documentation will also need updating if this change is made.
http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
Posted by "Mike Matrigali (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=comments#action_12359669 ]
Mike Matrigali commented on DERBY-737:
--------------------------------------
I don't see any downside to creating the statistics whether they exist or not. Not sure why the distinction was made.
First try at this might be just removing the if in opensource/java/engine/org/apache/derby/impl/sql/execute/AlterTableConstantAction.java!updateIndex():
if (td.statisticsExist(cd))
{
cCount = new CardinalityCounter(tc.openSortRowSource(sortIds[ind
ex]));
updateStatistics = true;
}
else
cCount = tc.openSortRowSource(sortIds[index]);
But life is probably not that easy. Likely there is slightly more work to create the statistics row vs. updating it. The work to insert the row can be
found in:
opensource/java/engine/org/apache/derby/impl/sql/execute/CreateIndexConstantAction.java
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Type: Improvement
> Components: SQL
> Versions: 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.1.1, 10.1.1.2, 10.1.2.0, 10.1.2.1, 10.2.0.0, 10.1.3.0, 10.1.2.2, 10.0.2.2
> Reporter: Kathey Marsden
> Priority: Minor
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Updated: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should
create statistics if they do not exist
Posted by "Mamta A. Satoor (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=all ]
Mamta A. Satoor updated DERBY-737:
----------------------------------
Derby Info: [Patch Available]
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.1.3.1, 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.2.1.6, 10.0.2.2
> Reporter: Kathey Marsden
> Assigned To: Mamta A. Satoor
> Priority: Minor
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE
should create statistics if they do not exist
Posted by "Mamta A. Satoor (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=comments#action_12442614 ]
Mamta A. Satoor commented on DERBY-737:
---------------------------------------
Thanks for taking the time out to review the patch, Bryan. Yes, the patch is ready for commit unless someone else is reviewing it too. Thanks again.
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.1.3.1, 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.2.1.6, 10.0.2.2
> Reporter: Kathey Marsden
> Assigned To: Mamta A. Satoor
> Priority: Minor
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE
should create statistics if they do not exist
Posted by "Mamta A. Satoor (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=comments#action_12440039 ]
Mamta A. Satoor commented on DERBY-737:
---------------------------------------
I am slightly confused by the 1st line in the description of this Jira entry. It says "There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE "
Shouldn't it read "There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be *updated* with SYSCS_UTIL.SYSCS_COMPRESS_TABLE "? The new sentence I am suggesting describes the current behavior of Derby which is that currently, SYSCS_UTIL.SYSCS_COMPRESS_TABLE can update statistics for indexes that already have rows in SYSSTATISTICS table. If there is no row for an index in SYSSTATISTICS table, then no statistics will be generated for that index.
With this Jira, we want to change Derby behavior such that if there is no row for an index in SYSSTATISTICS table, then at the time of SYSCS_UTIL.SYSCS_COMPRESS_TABLE, we should create a row for that index into SYSSTATISTICS table(provided that there is data in the table on which SYSCS_UTIL.SYSCS_COMPRESS_TABLE is getting run, no?) . If a row already exists for an index, then SYSCS_UTIL.SYSCS_COMPRESS_TABLE already does the job of updating the statistics for that index.
I am interested in working on this issue and want to be clear that I understand the current behavior and the expected new behavior.
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.1.1, 10.1.1.2, 10.1.2.0, 10.1.2.1, 10.2.1.6, 10.1.3.0, 10.1.2.2, 10.0.2.2
> Reporter: Kathey Marsden
> Priority: Minor
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Updated: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should
create statistics if they do not exist
Posted by "Mike Matrigali (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=all ]
Mike Matrigali updated DERBY-737:
---------------------------------
I believe your interpretation is correct. The request is to always "update" statistics when running the compress table command. Internally this may mean updating a row or creating a new row - the difference need not be documented to the user. Since the entire index is getting rebuilt, I can think of no reason not to gather the statistics and record them at this time.
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.1.1, 10.1.1.2, 10.1.2.0, 10.1.2.1, 10.2.1.6, 10.1.3.0, 10.1.2.2, 10.0.2.2
> Reporter: Kathey Marsden
> Priority: Minor
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Assigned: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should
create statistics if they do not exist
Posted by "Mamta A. Satoor (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=all ]
Mamta A. Satoor reassigned DERBY-737:
-------------------------------------
Assignee: Mamta A. Satoor
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.1.3.1, 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.2.1.6, 10.0.2.2
> Reporter: Kathey Marsden
> Assigned To: Mamta A. Satoor
> Priority: Minor
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE
should create statistics if they do not exist
Posted by "Mamta A. Satoor (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=comments#action_12442641 ]
Mamta A. Satoor commented on DERBY-737:
---------------------------------------
Bryan, thanks for committing the patch to the trunk. I think it will be useful to port this patch to prior releases because optimizer relies on the statistics information for indexes on a table. As for documentation, I will open another Jira entry for the doc changes.
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.1.3.1, 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.2.1.6, 10.0.2.2
> Reporter: Kathey Marsden
> Assigned To: Mamta A. Satoor
> Priority: Minor
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Closed: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should
create statistics if they do not exist
Posted by "Kathey Marsden (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-737?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kathey Marsden closed DERBY-737.
--------------------------------
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: https://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.0.2.0, 10.0.2.1, 10.0.2.2, 10.1.1.0, 10.1.2.1, 10.1.3.1, 10.2.1.6
> Reporter: Kathey Marsden
> Assigned To: Mamta A. Satoor
> Priority: Minor
> Fix For: 10.2.2.0, 10.3.0.0
>
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should
create statistics if they do not exist
Posted by "Kathey Marsden (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-737?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kathey Marsden updated DERBY-737:
---------------------------------
Fix Version/s: 10.1.3.2
updating fix version to reflect 10.1 fix
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: https://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.0.2.0, 10.0.2.1, 10.0.2.2, 10.1.1.0, 10.1.2.1, 10.1.3.1, 10.2.1.6
> Reporter: Kathey Marsden
> Assignee: Mamta A. Satoor
> Priority: Minor
> Fix For: 10.1.3.2, 10.2.2.0, 10.3.1.4
>
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should
create statistics if they do not exist
Posted by "Bryan Pendleton (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=all ]
Bryan Pendleton updated DERBY-737:
----------------------------------
Derby Info: (was: [Patch Available])
Committed the patch to subversion as revision 464551.
Mamta, do you think this patch needs to be ported to any prior releases?
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.1.3.1, 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.2.1.6, 10.0.2.2
> Reporter: Kathey Marsden
> Assigned To: Mamta A. Satoor
> Priority: Minor
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE
should create statistics if they do not exist
Posted by "Bryan Pendleton (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=comments#action_12442684 ]
Bryan Pendleton commented on DERBY-737:
---------------------------------------
The merge to the 10.2 branch was straightforward, and my 10.2 derbyall test run was clean. I propose to commit this merged change to the 10.2 branch.
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.1.3.1, 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.2.1.6, 10.0.2.2
> Reporter: Kathey Marsden
> Assigned To: Mamta A. Satoor
> Priority: Minor
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE
should create statistics if they do not exist
Posted by "Bryan Pendleton (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=comments#action_12442418 ]
Bryan Pendleton commented on DERBY-737:
---------------------------------------
Hi Mamta, I had a look at your patch.
Your changes seem good to me. Your new tests failed as expected without the code changes,
and passed as expected with the code changes. I also had a clean derbyall run with your changes.
Do you feel that this change is ready for commit? Is anyone else reviewing this change?
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.1.3.1, 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.2.1.6, 10.0.2.2
> Reporter: Kathey Marsden
> Assigned To: Mamta A. Satoor
> Priority: Minor
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Updated: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should
create statistics if they do not exist
Posted by "Mamta A. Satoor (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=all ]
Mamta A. Satoor updated DERBY-737:
----------------------------------
Attachment: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt
DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
I would like to submit a patch for this Jira entry. It is attached as DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt The changes have been very localized in AlterTableConstantAction.java!updateIndex() Currently, this method checks if statistics already exist for an index. If yes, then it sets a flag updateStatistics to true. Later, the code checks for this flag and drops the existing statistics and creates new statistics for that index provided the user table at this point is not empty. So, as we can see, if there is an index with no preexisting statistics, the flag updateStatistics will be set to false and hence no statistics related code is executed and hence even though the user table is not empty at the time of compress, no statistics get generated for such an index.
I am proposing to fix the problem by still using the flag to see if an index has pre-existing statistics. If yes,
then we should drop those statistics. Next, whether the index has pre-existing statistics or not, go ahead and
create new statistics for the index provided the user table is not currently empty.
I ran the derbyall suite on Windows XP with Sun JDK 1.4 and there were no new failures. In addition, I have added few tests to lang/compressTable.sql
Can someone please review this patch for me?
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.1.3.1, 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.2.1.6, 10.0.2.2
> Reporter: Kathey Marsden
> Priority: Minor
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE
should create statistics if they do not exist
Posted by "Mamta A. Satoor (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-737?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12573364#action_12573364 ]
Mamta A. Satoor commented on DERBY-737:
---------------------------------------
Merged changes into 10.1 codeline using revision 632065
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: https://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.0.2.0, 10.0.2.1, 10.0.2.2, 10.1.1.0, 10.1.2.1, 10.1.3.1, 10.2.1.6
> Reporter: Kathey Marsden
> Assignee: Mamta A. Satoor
> Priority: Minor
> Fix For: 10.2.2.0, 10.3.1.4
>
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (DERBY-737) SYSCS_UTIL.SYSCS_COMPRESS_TABLE should
create statistics if they do not exist
Posted by "Bryan Pendleton (JIRA)" <de...@db.apache.org>.
[ http://issues.apache.org/jira/browse/DERBY-737?page=all ]
Bryan Pendleton resolved DERBY-737.
-----------------------------------
Fix Version/s: 10.2.1.8
10.3.0.0
Resolution: Fixed
Merged the trunk fix to the 10.2 branch and committed to subversion
as revision 464683.
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they do not exist
> -----------------------------------------------------------------------------
>
> Key: DERBY-737
> URL: http://issues.apache.org/jira/browse/DERBY-737
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.1.3.1, 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.2.1.6, 10.0.2.2
> Reporter: Kathey Marsden
> Assigned To: Mamta A. Satoor
> Priority: Minor
> Fix For: 10.2.1.8, 10.3.0.0
>
> Attachments: DERBY737_v1_diff_SYSCS_COMPRESS_TABLE.txt, DERBY737_v1_stat_SYSCS_COMPRESS_TABLE.txt
>
>
> There must be an entry in the SYSSTATISTICS table in order for the cardinality statistics in SYSSTATISTICS to be created with SYSCS_UTIL.SYSCS_COMPRESS_TABLE
> SYSCS_UTIL.SYSCS_COMPRESS_TABLE should create statistics if they don't exist. The only way to create them if the index was created on an empty table, seems to be to drop and recreate the index after the table has been loaded.
> The documentation will also need updating if this change is made.
> http://db.apache.org/derby/docs/10.1/tuning/ctunstats57373.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira