<commit_before>/**
 *
 * Copyright 2004 The Apache Software Foundation
 *
 *  Licensed under the Apache License, Version 2.0 (the "License");
 *  you may not use this file except in compliance with the License.
 *  You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 *  Unless required by applicable law or agreed to in writing, software
 *  distributed under the License is distributed on an "AS IS" BASIS,
 *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 *  See the License for the specific language governing permissions and
 *  limitations under the License.
 */

package org.apache.geronimo.timer;

import javax.transaction.TransactionManager;

import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;

/**
 * @version $Rev$ $Date$
 */
public class TransactionalExecutorTask implements ExecutorTask {
    private static final Log log = LogFactory.getLog(TransactionalExecutorTask.class);

    private final Runnable userTask;
    private final WorkInfo workInfo;
    private final ThreadPooledTimer threadPooledTimer;

    private final TransactionManager transactionManager;
    private final int repeatCount;

    public TransactionalExecutorTask(Runnable userTask, WorkInfo workInfo, ThreadPooledTimer threadPooledTimer, TransactionManager transactionManager, int repeatCount) {
        this.userTask = userTask;
        this.workInfo = workInfo;
        this.threadPooledTimer = threadPooledTimer;
        this.transactionManager = transactionManager;
        this.repeatCount = repeatCount;
    }

    public void run() {
        for (int tries = 0; tries < repeatCount; tries++) {
            try {
                transactionManager.begin();
            } catch (Exception e) {
                log.warn("Exception occured while starting container transaction", e);
                break;
            }
            try {
                try {
                    userTask.run();
                } catch (Exception e) {
                    log.warn("Exception occured while running user task", e);
                }
                try {
                    threadPooledTimer.workPerformed(workInfo);
                } catch (PersistenceException e) {
                    log.warn("Exception occured while updating timer persistent state", e);
                }
            } finally {
                try {
                    transactionManager.commit();
                    if (workInfo.isOneTime()) {
                        threadPooledTimer.removeWorkInfo(workInfo);
                    }
                    // todo this is a very weird code structure.... returning from a finally is very confusing
                    return;
                } catch (Exception e) {
                    log.warn("Exception occured while completing container transaction", e);
                }
            }
        }
        if (workInfo.isOneTime()) {
            threadPooledTimer.removeWorkInfo(workInfo);
        }
        log.warn("Failed to execute work successfully");
    }

}<commit_msg>When tx is rollback only, don't try to commit and don't update the threadPooledTimer<commit_after>/**
 *
 * Copyright 2004 The Apache Software Foundation
 *
 *  Licensed under the Apache License, Version 2.0 (the "License");
 *  you may not use this file except in compliance with the License.
 *  You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 *  Unless required by applicable law or agreed to in writing, software
 *  distributed under the License is distributed on an "AS IS" BASIS,
 *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 *  See the License for the specific language governing permissions and
 *  limitations under the License.
 */

package org.apache.geronimo.timer;

import javax.transaction.TransactionManager;
import javax.transaction.Status;

import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;

/**
 * @version $Rev$ $Date$
 */
public class TransactionalExecutorTask implements ExecutorTask {
    private static final Log log = LogFactory.getLog(TransactionalExecutorTask.class);

    private final Runnable userTask;
    private final WorkInfo workInfo;
    private final ThreadPooledTimer threadPooledTimer;

    private final TransactionManager transactionManager;
    private final int repeatCount;

    public TransactionalExecutorTask(Runnable userTask, WorkInfo workInfo, ThreadPooledTimer threadPooledTimer, TransactionManager transactionManager, int repeatCount) {
        this.userTask = userTask;
        this.workInfo = workInfo;
        this.threadPooledTimer = threadPooledTimer;
        this.transactionManager = transactionManager;
        this.repeatCount = repeatCount;
    }

    public void run() {
        try {
            // try to do the work until it succeeded or we reach the repeat count
            boolean succeeded = false;
            for (int tries = 0; !succeeded && tries < repeatCount; tries++) {
                try {
                    if (!beginWork()) {
                        break;
                    }

                    work();
                } finally {
                    succeeded = completeWork();
                }
            }

            // if this was a one time thing, remove the job
            if (workInfo.isOneTime()) {
                threadPooledTimer.removeWorkInfo(workInfo);
            }

            // if we didn't succeed, log it
            if (!succeeded) {
                log.warn("Failed to execute work successfully");
            }
        } catch (RuntimeException e) {
            log.warn("RuntimeException occured while running user task", e);
            throw e;
        } catch (Error e) {
            log.warn("Error occured while running user task", e);
            throw e;
        }
    }

    private boolean beginWork() {
        try {
            transactionManager.begin();
        } catch (Exception e) {
            log.warn("Exception occured while starting container transaction", e);
            return false;
        }
        return true;
    }

    private void work() {
        try {
            userTask.run();
        } catch (Exception e) {
            log.warn("Exception occured while running user task", e);
        }
    }

    private boolean completeWork() {
        try {
            if (transactionManager.getStatus() == Status.STATUS_ACTIVE) {
                // clean up the work persistent data
                try {
                    threadPooledTimer.workPerformed(workInfo);
                } catch (PersistenceException e) {
                    log.warn("Exception occured while updating timer persistent state", e);
                }

                // commit the tx
                transactionManager.commit();

                // all is cool
                return true;
            } else {
                // tx was marked rollback, so roll it back
                transactionManager.rollback();
            }
        } catch (Exception e) {
            log.warn("Exception occured while completing container transaction", e);
        }
        // something bad happened
        return false;
    }

}
{'fraction_non_alphanumeric': 0.07627702051493873, 'fraction_numerical': 0.0024783147459727386, 'mean_word_length': 1.9673202614379084, 'pattern_counts': {'":': 0, '<': 5, '<?xml version=': 0, '>': 3, 'https://': 0, 'lorem ipsum': 0, 'www.': 2, 'xml': 0}, 'pii_count': 0, 'substrings_counts': 0, 'word_list_counts': {'cursed_substrings.json': 1, 'profanity_word_list.json': 0, 'sexual_word_list.json': 0, 'zh_pornsignals.json': 0}}
{'dir': 'git-commits-cleaned', 'id': '6039266', 'n_tokens_mistral': 1756, 'n_tokens_neox': 1678, 'n_words': 581}