Added support for legacy alter table flag#848
Conversation
|
hi, thanks for the PR. Can you run |
| public void setLegacyAlterTableFlag(boolean use) { | ||
| set(Pragma.LEGACY_ALTER_TABLE, use); | ||
| } |
There was a problem hiding this comment.
| public void setLegacyAlterTableFlag(boolean use) { | |
| set(Pragma.LEGACY_ALTER_TABLE, use); | |
| } | |
| public void setLegacyAlterTable(boolean flag) { | |
| set(Pragma.LEGACY_ALTER_TABLE, flag); | |
| } |
| public void setLegacyAlterTableFlag(boolean use) { | ||
| config.setLegacyAlterTableFlag(use); | ||
| } |
There was a problem hiding this comment.
| public void setLegacyAlterTableFlag(boolean use) { | |
| config.setLegacyAlterTableFlag(use); | |
| } | |
| public void setLegacyAlterTable(boolean flag) { | |
| config.setLegacyAlterTable(flag); | |
| } |
| */ | ||
| public void setLegacyAlterTableFlag(boolean use) { | ||
| set(Pragma.LEGACY_ALTER_TABLE, use); | ||
| public void setLegacyAlterTableFlag(boolean flag) { |
There was a problem hiding this comment.
can you also change the name from setLegacyAlterTableFlag to setLegacyAlterTable to be consistent with other methods?
There was a problem hiding this comment.
my bad didn't spot the method rename
| */ | ||
| public void setLegacyAlterTableFlag(boolean use) { | ||
| config.setLegacyAlterTableFlag(use); | ||
| public void setLegacyAlterTableFlag(boolean flag) { |
There was a problem hiding this comment.
can you also change the name from setLegacyAlterTableFlag to setLegacyAlterTable to be consistent with other methods?
|
thanks for your contribution! |
|
thank you for reviewing! Can you tell me the expected date of release? |
no idea, there's a few PRs in flight to review. The snapshot is already available, you can try that. |
3 participants
Added support for legacy alter table flag
More info here