Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated "Change Set" wording to "Changeset" #3042

Merged
merged 2 commits into from Jul 12, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
16 changes: 8 additions & 8 deletions liquibase-core/src/main/java/liquibase/Liquibase.java
Expand Up @@ -495,7 +495,7 @@ public void run() throws Exception {

//
// Create an iterator which will be used with a ListVisitor
// to grab the list of change sets for the update
// to grab the list of changesets for the update
//
ChangeLogIterator listLogIterator = new ChangeLogIterator(changeLog,
new ShouldRunChangeSetFilter(database),
Expand Down Expand Up @@ -612,7 +612,7 @@ public void run() throws Exception {

//
// Create an iterator which will be used with a ListVisitor
// to grab the list of change sets for the update
// to grab the list of changesets for the update
//
List<RanChangeSet> ranChangeSetList = database.getRanChangeSetList();
ChangeLogIterator listLogIterator = new ChangeLogIterator(changeLog,
Expand Down Expand Up @@ -691,7 +691,7 @@ public void run() throws Exception {
/* We have no other choice than to save the current Executer here. */
@SuppressWarnings("squid:S1941")
Executor oldTemplate = getAndReplaceJdbcExecutor(output);
outputHeader("Update " + changesToApply + " Change Sets Database Script");
outputHeader("Update " + changesToApply + " Changesets Database Script");

update(changesToApply, contexts, labelExpression);

Expand Down Expand Up @@ -868,7 +868,7 @@ public void run() throws Exception {

//
// Create an iterator which will be used with a ListVisitor
// to grab the list of change sets for the update
// to grab the list of changesets for the update
//
ChangeLogIterator listLogIterator = new ChangeLogIterator(database.getRanChangeSetList(), changeLog,
new AlreadyRanChangeSetFilter(database.getRanChangeSetList()),
Expand Down Expand Up @@ -1140,7 +1140,7 @@ public void run() throws Exception {

//
// Create an iterator which will be used with a ListVisitor
// to grab the list of change sets for the update
// to grab the list of changesets for the update
//
List<RanChangeSet> ranChangeSetList = database.getRanChangeSetList();
ChangeLogIterator listLogIterator = new ChangeLogIterator(ranChangeSetList, changeLog,
Expand Down Expand Up @@ -1308,7 +1308,7 @@ public void run() throws Exception {

//
// Create an iterator which will be used with a ListVisitor
// to grab the list of change sets for the update
// to grab the list of changesets for the update
//
List<RanChangeSet> ranChangeSetList = database.getRanChangeSetList();
ChangeLogIterator listLogIterator = new ChangeLogIterator(ranChangeSetList, changeLog,
Expand Down Expand Up @@ -1466,7 +1466,7 @@ public void run() throws Exception {

//
// Create an iterator which will be used with a ListVisitor
// to grab the list of change sets for the update
// to grab the list of changesets for the update
//
ChangeLogIterator listLogIterator = buildChangeLogIterator(tag, changeLog, contexts, labelExpression);

Expand Down Expand Up @@ -2056,7 +2056,7 @@ public void reportStatus(boolean verbose, Contexts contexts, LabelExpression lab
out.append(StreamUtil.getLineSeparator());
} else {
out.append(String.valueOf(unrunChangeSets.size()));
out.append(" change sets have not been applied to ");
out.append(" changesets have not been applied to ");
out.append(getDatabase().getConnection().getConnectionUserName());
out.append("@");
out.append(getDatabase().getConnection().getURL());
Expand Down
Expand Up @@ -484,7 +484,7 @@ public SqlStatement[] generateStatements(Database database) {
} catch (UnexpectedLiquibaseException ule) {
if ((getChangeSet() != null) && (getChangeSet().getFailOnError() != null) && !getChangeSet()
.getFailOnError()) {
LOG.info("Change set " + getChangeSet().toString(false) +
LOG.info("Changeset " + getChangeSet().toString(false) +
" failed, but failOnError was false. Error: " + ule.getMessage());
return new SqlStatement[0];
} else {
Expand Down
Expand Up @@ -28,7 +28,7 @@
" want to create your own custom refactoring class.\n" +
"\n" +
"To create your own custom refactoring, simply create a class that implements the liquibase.change.custom.CustomSqlChange " +
"or liquibase.change.custom.CustomTaskChange interface and use the <custom> tag in your change set.\n" +
"or liquibase.change.custom.CustomTaskChange interface and use the <custom> tag in your changeset.\n" +
"\n" +
"If your change can be rolled back, implement the liquibase.change.custom.CustomSqlRollback interface as well.\n" +
"\n" +
Expand Down
Expand Up @@ -9,7 +9,7 @@
* implementing this interface is preferred over CustomTaskChange because the SQL can either be executed
* directly or saved to a text file for later use depending on the migration mode used. To allow
* the change to be rolled back, also implement the CustomSqlRollback interface. If your change requires sql-based
* logic and non-sql-based logic, it is best to create a change set that contains a mix of CustomSqlChange and CustomTaskChange calls.
* logic and non-sql-based logic, it is best to create a changeset that contains a mix of CustomSqlChange and CustomTaskChange calls.
*
* @see liquibase.change.custom.CustomSqlRollback
* @see liquibase.change.custom.CustomTaskChange
Expand Down
Expand Up @@ -8,7 +8,7 @@
* If you are updating a database through SQL, implementing CustomSqlChange is preferred because the SQL can either be executed
* directly or saved to a text file for later use depending on the migration mode used.
* To allow the change to be rolled back, also implement the CustomTaskRollback interface.
* If your change requires sql-based logic and non-sql-based logic, it is best to create a change set that contains a mix of CustomSqlChange and CustomTaskChange calls.
* If your change requires sql-based logic and non-sql-based logic, it is best to create a changeset that contains a mix of CustomSqlChange and CustomTaskChange calls.
*
* @see liquibase.change.custom.CustomTaskRollback
* @see liquibase.change.custom.CustomSqlChange
Expand Down
Expand Up @@ -4,7 +4,7 @@
* <p>
* To create your own custom refactoring, simply create a class that implements the
* {@link liquibase.change.custom.CustomSqlChange} or {@link liquibase.change.custom.CustomTaskChange} interface and
* use the &lt;custom&gt; tag in your change set.<p>
* use the &lt;custom&gt; tag in your changeset.<p>
* <p>
* If your change can be rolled back, implement the {@link liquibase.change.custom.CustomSqlRollback} interface as
* well.<p>
Expand Down
Expand Up @@ -100,7 +100,7 @@ public Date getRanDate(ChangeSet changeSet) throws DatabaseException, DatabaseHi

/**
*
* Return the last deployment ID from the change sets that have been run or null
* Return the last deployment ID from the changesets that have been run or null
*
* @return String
* @throws DatabaseException
Expand Down
Expand Up @@ -93,7 +93,7 @@ public void run() throws Exception {
Scope.child(Scope.Attr.changeSet.name(), changeSet, () -> {
if (finalShouldVisit && !alreadySaw(changeSet)) {
//
// Go validate any change sets with an Executor if
// Go validate any changesets with an Executor if
// we are using a ValidatingVisitor
//
if (visitor instanceof ValidatingVisitor) {
Expand All @@ -103,7 +103,7 @@ public void run() throws Exception {
//
// Execute the visit call in its own scope with a new
// CompositeLogService and BufferLogService in order
// to capture the logging for just this change set. The
// to capture the logging for just this changeset. The
// log is sent to Hub if available
//
Map<String, Object> values = new HashMap<>();
Expand Down Expand Up @@ -131,8 +131,8 @@ public void run() throws Exception {


//
// Make sure that any change set which has a runWith=<executor> setting
// has a valid Executor, and that the changes in the change set
// Make sure that any changeset which has a runWith=<executor> setting
// has a valid Executor, and that the changes in the changeset
// are eligible for execution by this Executor
//
private void validateChangeSetExecutor(ChangeSet changeSet, RuntimeEnvironment env) throws LiquibaseException {
Expand Down
Expand Up @@ -477,7 +477,7 @@ protected void handleRollbackNode(ParsedNode rollbackNode, ResourceAccessor reso
}
}
if (changeSet == null) {
throw new ParsedNodeException("Change set " + new ChangeSet(changeSetId, changeSetAuthor, false, false, changeSetPath, null, null, null).toString(false) + " does not exist");
throw new ParsedNodeException("Changeset " + new ChangeSet(changeSetId, changeSetAuthor, false, false, changeSetPath, null, null, null).toString(false) + " does not exist");
}
for (Change change : changeSet.getChanges()) {
rollback.getChanges().add(change);
Expand Down Expand Up @@ -539,7 +539,7 @@ public ExecType execute(DatabaseChangeLog databaseChangeLog, Database database)
* This method will actually execute each of the changes in the list against the
* specified database.
*
* @return should change set be marked as ran
* @return should changeset be marked as ran
*/
public ExecType execute(DatabaseChangeLog databaseChangeLog, ChangeExecListener listener, Database database)
throws MigrationFailedException {
Expand Down Expand Up @@ -691,7 +691,7 @@ public ExecType execute(DatabaseChangeLog databaseChangeLog, ChangeExecListener
throw new MigrationFailedException(this, e);
}
if ((getFailOnError() != null) && !getFailOnError()) {
log.info("Change set " + toString(false) + " failed, but failOnError was false. Error: " + e.getMessage());
log.info("Changeset " + toString(false) + " failed, but failOnError was false. Error: " + e.getMessage());
log.fine("Failure Stacktrace", e);
execType = ExecType.FAILED;
} else {
Expand Down Expand Up @@ -946,7 +946,7 @@ public Collection<Labels> getInheritableLabels() {

/**
*
* Build and return a string which contains both the change set and inherited context
* Build and return a string which contains both the changeset and inherited context
*
* @return String
*
Expand All @@ -967,7 +967,7 @@ public String buildFullContext() {

/**
*
* Build and return a string which contains both the change set and inherited labels
* Build and return a string which contains both the changeset and inherited labels
*
* @return String
*
Expand Down
Expand Up @@ -66,7 +66,7 @@ public void setComments(String comments) {
}

/**
* Will the change set run next time.
* Will the changeset run next time.
*/
public boolean getWillRun() {
return willRun;
Expand All @@ -77,7 +77,7 @@ public void setWillRun(boolean willRun) {
}

/**
* Reasons the change set will or will not run next time. Returns empty set if no reasons were given
* Reasons the changeset will or will not run next time. Returns empty set if no reasons were given
*/
public Set<ChangeSetFilterResult> getFilterResults() {
return filterResults;
Expand All @@ -88,7 +88,7 @@ public void setFilterResults(Set<ChangeSetFilterResult> filterResults) {
}

/**
* Convenience method to check wither a given ChangeSetFilter type is a reason for running the change set or not.
* Convenience method to check wither a given ChangeSetFilter type is a reason for running the changeset or not.
*/
public boolean isFilteredBy(Class<? extends ChangeSetFilter> filterType) {
if (!willRun) {
Expand All @@ -108,7 +108,7 @@ public boolean isFilteredBy(Class<? extends ChangeSetFilter> filterType) {
}

/**
* Return the checksum stored from the last execution of the change set. Returns null if it has not ran before
* Return the checksum stored from the last execution of the changeset. Returns null if it has not ran before
*/
public CheckSum getStoredCheckSum() {
return storedCheckSum;
Expand All @@ -119,7 +119,7 @@ public void setStoredCheckSum(CheckSum storedCheckSum) {
}

/**
* Return the date the change set was last executed. Returns null if it has not ran before
* Return the date the changeset was last executed. Returns null if it has not ran before
*/
public Date getDateLastExecuted() {
return dateLastExecuted;
Expand All @@ -130,7 +130,7 @@ public void setDateLastExecuted(Date dateLastExecuted) {
}

/**
* Returns true if the change set was ran previously.
* Returns true if the changeset was ran previously.
*/
public boolean getPreviouslyRan() {
return previouslyRan;
Expand Down
Expand Up @@ -7,7 +7,7 @@
import java.util.Date;

/**
* Encapsulates information about a previously-ran change set. Used to build rollback statements.
* Encapsulates information about a previously-ran changeset. Used to build rollback statements.
*/
public class RanChangeSet {
private final String changeLog;
Expand Down
Expand Up @@ -16,9 +16,9 @@ public ChangeSetFilterResult accepts(ChangeSet changeSet) {
RanChangeSet ranChangeSet = getRanChangeSet(changeSet);
if ((ranChangeSet != null) && ((ranChangeSet.getExecType() == null) || ranChangeSet.getExecType().equals
(ChangeSet.ExecType.EXECUTED) || ranChangeSet.getExecType().equals(ChangeSet.ExecType.RERAN))) {
return new ChangeSetFilterResult(true, "Change set was executed previously", this.getClass());
return new ChangeSetFilterResult(true, "Changeset was executed previously", this.getClass());
} else {
return new ChangeSetFilterResult(false, "Change set was not previously executed", this.getClass());
return new ChangeSetFilterResult(false, "Changeset was not previously executed", this.getClass());
}
}
}
Expand Up @@ -42,9 +42,9 @@ private String changeLogToString(String id, String author, String changeLog) {
@Override
public ChangeSetFilterResult accepts(ChangeSet changeSet) {
if (changeLogsAfterTag.contains(changeLogToString(changeSet.getId(), changeSet.getAuthor(), changeSet.getFilePath()))) {
return new ChangeSetFilterResult(true, "Change set is before tag '"+tag+"'", this.getClass());
return new ChangeSetFilterResult(true, "Changeset is before tag '"+tag+"'", this.getClass());
} else {
return new ChangeSetFilterResult(false, "Change set after tag '"+tag+"'", this.getClass());
return new ChangeSetFilterResult(false, "Changeset after tag '"+tag+"'", this.getClass());
}
}
}
Expand Up @@ -14,9 +14,9 @@ public AlreadyRanChangeSetFilter(List<RanChangeSet> ranChangeSets) {
@Override
public ChangeSetFilterResult accepts(ChangeSet changeSet) {
if (getRanChangeSet(changeSet) != null) {
return new ChangeSetFilterResult(true, "Change set already ran", this.getClass());
return new ChangeSetFilterResult(true, "Changeset already ran", this.getClass());
} else {
return new ChangeSetFilterResult(false, "Change set has not ran", this.getClass());
return new ChangeSetFilterResult(false, "Changeset has not ran", this.getClass());
}
}

Expand Down
Expand Up @@ -18,7 +18,7 @@ public ChangeSetFilterResult(boolean accepted, String message, Class<? extends C
}

/**
* Was the change set accepted by the filter
* Was the changeset accepted by the filter
*/
public boolean isAccepted() {
return accepted;
Expand All @@ -32,7 +32,7 @@ public String getMessage() {
}

/**
* Returns the class of the filter that accepted or rejected this change set.
* Returns the class of the filter that accepted or rejected this changeset.
*/
public Class<? extends ChangeSetFilter> getFilter() {
return filter;
Expand Down
Expand Up @@ -36,7 +36,7 @@ public ChangeSetFilterResult accepts(ChangeSet changeSet) {

Collection<ContextExpression> inheritableContexts = changeSet.getInheritableContexts();
if (changeSet.getContexts().isEmpty() && inheritableContexts.isEmpty()) {
return new ChangeSetFilterResult(true, "Change set runs under all contexts", this.getClass());
return new ChangeSetFilterResult(true, "Changeset runs under all contexts", this.getClass());
}

if (changeSet.getContexts().matches(contexts) && ContextExpression.matchesAll(inheritableContexts, contexts)) {
Expand Down
Expand Up @@ -16,9 +16,9 @@ public ChangeSetFilterResult accepts(ChangeSet changeSet) {
changeSetsSeen++;

if (changeSetsSeen <= changeSetsToAllow) {
return new ChangeSetFilterResult(true, "One of "+changeSetsToAllow+" change sets to run", this.getClass());
return new ChangeSetFilterResult(true, "One of "+changeSetsToAllow+" changesets to run", this.getClass());
} else {
return new ChangeSetFilterResult(false, "Only running "+changeSetsToAllow+" change sets", this.getClass());
return new ChangeSetFilterResult(false, "Only running "+changeSetsToAllow+" changesets", this.getClass());
}
}
}
Expand Up @@ -30,9 +30,9 @@ private String changeLogToString(String id, String author, String changeLog) {
@Override
public ChangeSetFilterResult accepts(ChangeSet changeSet) {
if (changeLogsAfterDate.contains(changeLogToString(changeSet.getId(), changeSet.getAuthor(), changeSet.getFilePath()))) {
return new ChangeSetFilterResult(true, "Change set ran after "+ new ISODateFormat().format(new java.sql.Timestamp(date.getTime())), this.getClass());
return new ChangeSetFilterResult(true, "Changeset ran after "+ new ISODateFormat().format(new java.sql.Timestamp(date.getTime())), this.getClass());
} else {
return new ChangeSetFilterResult(false, "Change set ran before "+ new ISODateFormat().format(new java.sql.Timestamp(date.getTime())), this.getClass());
return new ChangeSetFilterResult(false, "Changeset ran before "+ new ISODateFormat().format(new java.sql.Timestamp(date.getTime())), this.getClass());
}
}
}
}
Expand Up @@ -5,13 +5,13 @@
public class IgnoreChangeSetFilter implements ChangeSetFilter {
public ChangeSetFilterResult accepts(ChangeSet changeSet) {
if (changeSet.isIgnore()) {
return new ChangeSetFilterResult(false, "Change set is ignored", this.getClass());
return new ChangeSetFilterResult(false, "Changeset is ignored", this.getClass());
}
else {
if (changeSet.isInheritableIgnore()) {
return new ChangeSetFilterResult(false, "Change set is ignored", this.getClass());
return new ChangeSetFilterResult(false, "Changeset is ignored", this.getClass());
}
}
return new ChangeSetFilterResult(true, "Change set is not ignored", this.getClass());
return new ChangeSetFilterResult(true, "Changeset is not ignored", this.getClass());
}
}