我应该提交以下代码吗？

Question

我的代码：

122 #
123 my $hfpDbh = undef;
124 unless (
125         $hfpDbh = DBI->connect("dbi:Pg:host=....")#removed something
128 ) {
129         Log( ERROR, "" );
130         Exit( 1 )
131 }
132 $hfpDbh->{RaiseError} = 1;
133 $hfpDbh->{AutoCommit} = 0;
134 
135 ( my $mydata, $msg ) = load_data( $hfpDbh, $DatFile );
136 unless ( defined($mydata) )
137 {
138         Log(INFO, "Calling exit...2");
139 }
140 #$hfpDbh->disconnect();
141 Exit( 0 );
142 Log(INFO, "Calling exit...4");
143 
144 
145 sub load_data
146 {
147         my ( $dbh, $DatFile ) = @_;
148         my $msg = '';
149         unless ( $dbh ) {
150                 $msg = 'cannot load data, no DB handle';
151                 Log( ERROR, $msg );
152         }
153         Log(INFO, "Call load_data...");
154         my $q = "SELECT ip as ip FROM rules WHERE active = 'true' AND isGood = 'true';";
155         my $stmt = undef;
156         unless ( $stmt = $dbh->prepare( $q ) ) {
157                 $msg = "unable to prepare SQL query: $q";
158                 Log( ERROR, $msg );
159         }
160 
161         eval { $stmt->execute() };
162         if ( $@ ) {
163                 $msg = "failed to execute SQL query: $@";
164                 Log( ERROR, $msg );
165         }
166 
167         my $data = {};
168         while ( my $row = $stmt->fetchrow_hashref() ) {
169                 #Log(INFO, "testing row");
170         }
171         $stmt->finish();
172         return $data, $msg;
173 }

警告是：

Issuing rollback() due to DESTROY without explicit disconnect() of DBD::Pg::db handle

如果我在第 171 行之后添加“$dbh->commit()”，上面的警告就消失了。

如果我没有在第 171 行之后添加“$dbh->commit()”，而是调用了“$hfpDbh->disconnect();”在第 140 行，上面的警告也消失了。

我的问题是： 警告表示有未提交的事务？这就是为什么我需要明确提交或断开连接以修复警告。但是代码中只有 SELECT 操作。我错过了什么？

谢谢。

Answer 1

• 由于您根本没有修改数据库，因此不需要启用事务，因此不要将AutoCommit 设置为零。这样也不需要在任何地方调用commit，当句柄超出dcope时数据库会自动断开

• 由于您自己处理错误，因此不应将 RaiseError 设置为 1。如果发生任何错误，这将导致您的程序立即终止，并且您自己的处理代码不会被执行

• 无需致电finish。它在这里不会造成任何伤害，但它也毫无意义，几乎永远不需要

Answer 2

警告表示有未提交的事务？

自从您请求使用事务以来，有一个未提交的事务，但警告实际上通知您隐式执行了回滚。它会告诉您这一点，因为这可能会导致信息丢失。显然，在这种情况下，它不会导致信息丢失，但支票不够聪明，无法意识到这一点。

我错过了什么？

$hfpDbh->disconnect(); 或 $hfpDbh->rollback();