db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Satheesh Bandaram <sathe...@Sourcery.Org>
Subject Re: [PATCH] Updatable Resultset JDBC api (update and delete for forward only updatable resultsets) support for Network Server
Date Wed, 08 Jun 2005 01:38:26 GMT
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
<body bgcolor="#ffffff" text="#000000">
Sending&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java\client\org\apache\derby\client\am\ResultSet.java<br>
Transmitting file data .<br>
Committed revision 189486.<br>
Mamta Satoor wrote:<br>
<blockquote cite="midd9619e4a050604084774392791@mail.gmail.com"
  <div>Hi Satheesh,</div>
  <div>I have a very small patch to address&nbsp;your review&nbsp;comments on
Updatable Resultset support for Network Server using Derby Net Client.</div>
  <div>svn stat </div>
  <div>M&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java\client\org\apache\derby\client\am\ResultSet.java</div>
  <div>Please commit it if there are no issues.</div>
  <div><span class="gmail_quote">On 5/23/05, <b
 class="gmail_sendername">Satheesh Bandaram</b> &lt;<a
  <blockquote class="gmail_quote"
 style="border-left: 1px solid rgb(204, 204, 204); margin: 0px 0px 0px 0.8ex; padding-left:
minor comments... Use them if they apply....<br>
      <li>Not sure if resultSetMetaData_.sqlxRdbnam_[baseTableColumn]
is ever set, so no need to check for that. </li>
      <li>Use sqlxBasename_ instead of sqlxSchema_ when looking for a
valid base table. While both may work, it may be more readable. </li>
 cite="http://midd9619e4a05052102445330a565@mail.gmail.com" type="cite">
  <pre wrap="">
<hr size="4" width="90%">
Index: java/client/org/apache/derby/client/am/ResultSet.java
--- java/client/org/apache/derby/client/am/ResultSet.java	(revision 179793)
+++ java/client/org/apache/derby/client/am/ResultSet.java	(working copy)
@@ -3134,7 +3134,7 @@
         int baseTableColumn = 0;
         int totalColumns = resultSetMetaData_.getColumnCount();
         for (; baseTableColumn &lt; totalColumns; baseTableColumn++) {
-            if (resultSetMetaData_.sqlxSchema_[baseTableColumn] != null)
+            if (resultSetMetaData_.sqlxBasename_[baseTableColumn] != null)
@@ -3142,11 +3142,6 @@
         if (baseTableColumn == totalColumns)
             baseTableColumn = 0;
-        if (resultSetMetaData_.sqlxRdbnam_[baseTableColumn] != null &amp;&amp;
-                !resultSetMetaData_.sqlxRdbnam_[baseTableColumn].equals(""))      // catalog
-        {
-            tableName += "\"" + resultSetMetaData_.sqlxRdbnam_[baseTableColumn] + "\".";
-        }
         //dervied column like select 2 from t1, has null schema and table name
         if (resultSetMetaData_.sqlxSchema_[baseTableColumn] != null &amp;&amp; !resultSetMetaData_.sqlxSchema_[baseTableColumn].equals(""))
             tableName += "\"" + resultSetMetaData_.sqlxSchema_[baseTableColumn] + "\".";

View raw message