[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ekaterina Dimitrova updated CASSANDRA-17181: Fix Version/s: 4.0.2 4.1 (was: 4.x) (was: 4.0.x) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.0.2, 4.1 > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Lerer updated CASSANDRA-17181: --- Source Control Link: https://github.com/apache/cassandra/commit/75482d0a8ccd0b0d370aeb7ee60c72cd47a191b0 Resolution: Fixed Status: Resolved (was: Ready to Commit) Patch committed into trunk at 75482d0a8ccd0b0d370aeb7ee60c72cd47a191b0 > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.0.x, 4.x > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Lerer updated CASSANDRA-17181: --- Status: Ready to Commit (was: Changes Suggested) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.0.x, 4.x > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bartlomiej updated CASSANDRA-17181: --- Attachment: CASSANDRA-17181-4.0.patch > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.0.x, 4.x > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bartlomiej updated CASSANDRA-17181: --- Attachment: (was: CASSANDRA-17181-4.0.patch) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.0.x, 4.x > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ekaterina Dimitrova updated CASSANDRA-17181: Status: Changes Suggested (was: Review In Progress) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.0.x, 4.x > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ekaterina Dimitrova updated CASSANDRA-17181: Fix Version/s: 4.0.x 4.x (was: 4.1) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.0.x, 4.x > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ekaterina Dimitrova updated CASSANDRA-17181: Reviewers: Benjamin Lerer, Ekaterina Dimitrova (was: Benjamin Lerer) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.1 > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Lerer updated CASSANDRA-17181: --- Status: Review In Progress (was: Patch Available) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.1 > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Lerer updated CASSANDRA-17181: --- Test and Documentation Plan: The patch update existing tests Status: Patch Available (was: Open) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.1 > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Lerer updated CASSANDRA-17181: --- Workflow: Copy of Cassandra Default Workflow (was: Copy of Cassandra Bug Workflow) Issue Type: Improvement (was: Bug) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Improvement > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.1 > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bartlomiej updated CASSANDRA-17181: --- Attachment: CASSANDRA-17181-4.0.patch > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Bug > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.1 > > Attachments: CASSANDRA-17181-4.0.patch > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Lerer updated CASSANDRA-17181: --- Description: {{SchemaCQLHelperTest}} is used during a snapshot to generate the {{schema.cql}} file. The methods accept the following paramaters: {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. Those parameters are in practice always set to true by the calling code and therefore can be removed. was: When a schema.cql is created during a snapshot, {{CREATE TYPE}} statements are not created with {{IF NOT EXISTS}} as the {{CREATE TABLE}} statements. This can cause some issues when using those statements during restore. Additional Information for newcomers: In 3.0 and 3.11 the code use to generate the {{schema.cql}} file is in {{TableCQLHelper}} and the unit tests in {{TableCQLHelperTest}}. In 4.0 the bug does not exist anymore but the test {{SchemaCQLHelperTest}} does not test the correct behavior and the code could be simplified > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Bug > Components: Local/Snapshots >Reporter: Benjamin Lerer >Assignee: Bartlomiej >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.1 > > > {{SchemaCQLHelperTest}} is used during a snapshot to generate the > {{schema.cql}} file. The methods accept the following paramaters: > {{includeDroppedColumns}}, {{internals}} and {{ifNotExists}}. > Those parameters are in practice always set to true by the calling code and > therefore can be removed. -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Lerer updated CASSANDRA-17181: --- Fix Version/s: 4.1 (was: 3.0.x) (was: 3.11.x) (was: 4.0.x) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Bug > Components: Local/Snapshots >Reporter: Benjamin Lerer >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 4.1 > > > When a schema.cql is created during a snapshot, {{CREATE TYPE}} statements > are not created with {{IF NOT EXISTS}} as the {{CREATE TABLE}} statements. > This can cause some issues when using those statements during restore. > Additional Information for newcomers: > In 3.0 and 3.11 the code use to generate the {{schema.cql}} file is in > {{TableCQLHelper}} and the unit tests in {{TableCQLHelperTest}}. > In 4.0 the bug does not exist anymore but the test {{SchemaCQLHelperTest}} > does not test the correct behavior and the code could be simplified -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Updated] (CASSANDRA-17181) SchemaCQLHelperTest methods can be simplified
[ https://issues.apache.org/jira/browse/CASSANDRA-17181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Lerer updated CASSANDRA-17181: --- Summary: SchemaCQLHelperTest methods can be simplified (was: schema.cql should have IF NOT EXISTS for CREATE TYPE) > SchemaCQLHelperTest methods can be simplified > - > > Key: CASSANDRA-17181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17181 > Project: Cassandra > Issue Type: Bug > Components: Local/Snapshots >Reporter: Benjamin Lerer >Priority: Normal > Labels: AdventCalendar2021, lhf > Fix For: 3.0.x, 3.11.x, 4.0.x > > > When a schema.cql is created during a snapshot, {{CREATE TYPE}} statements > are not created with {{IF NOT EXISTS}} as the {{CREATE TABLE}} statements. > This can cause some issues when using those statements during restore. > Additional Information for newcomers: > In 3.0 and 3.11 the code use to generate the {{schema.cql}} file is in > {{TableCQLHelper}} and the unit tests in {{TableCQLHelperTest}}. > In 4.0 the bug does not exist anymore but the test {{SchemaCQLHelperTest}} > does not test the correct behavior and the code could be simplified -- This message was sent by Atlassian Jira (v8.20.1#820001) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org