]> granicus.if.org Git - postgresql/commitdiff
Fix for indexing problems.
authorBruce Momjian <bruce@momjian.us>
Wed, 2 Sep 1998 23:05:37 +0000 (23:05 +0000)
committerBruce Momjian <bruce@momjian.us>
Wed, 2 Sep 1998 23:05:37 +0000 (23:05 +0000)
src/backend/access/index/indexam.c
src/backend/catalog/indexing.c
src/backend/commands/vacuum.c
src/backend/parser/parse_target.c
src/backend/storage/ipc/ipc.c
src/backend/tcop/pquery.c
src/backend/utils/misc/trace.c

index 4c2289ce16052d354c458f9a5e4ebad761afab45..04445f14dd0c5c15650233cedb81168db3682d87 100644 (file)
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/access/index/indexam.c,v 1.25 1998/09/01 04:26:55 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/access/index/indexam.c,v 1.26 1998/09/02 23:05:21 momjian Exp $
  *
  * INTERFACE ROUTINES
  *             index_open              - open an index relation by relationId
@@ -190,23 +190,7 @@ index_insert(Relation relation,
        specificResult = (InsertIndexResult)
                fmgr(procedure, relation, datum, nulls, heap_t_ctid, heapRel, NULL);
 
-       /* ----------------
-        *      the insert proc is supposed to return a "specific result" and
-        *      this routine has to return a "general result" so after we get
-        *      something back from the insert proc, we allocate a
-        *      "general result" and copy some crap between the two.
-        *
-        *      As far as I'm concerned all this result shit is needlessly c
-        *      omplicated and should be eliminated.  -cim 1/19/91
-        *
-        *      mao concurs.  regardless of how we feel here, however, it is
-        *      important to free memory we don't intend to return to anyone.
-        *      2/28/91
-        *
-        *      this "general result" crap is now gone. -ay 3/6/95
-        * ----------------
-        */
-
+       /* must be pfree'ed */
        return specificResult;
 }
 
index f685bf3e7958b4361e83b3fb8f887752a5ebf884..ad31265f501fc12f1df7f59abe49bab47c62d62f 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/catalog/indexing.c,v 1.29 1998/09/01 16:21:47 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/catalog/indexing.c,v 1.30 1998/09/02 23:05:23 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -109,12 +109,12 @@ CatalogIndexInsert(Relation *idescs,
        HeapTuple       index_tup;
        TupleDesc       heapDescriptor;
        Form_pg_index index_form;
-       Datum           datum;
+       Datum           datum[INDEX_MAX_KEYS];
+       char            nulls[INDEX_MAX_KEYS];
        int                     natts;
        AttrNumber *attnumP;
        FuncIndexInfo finfo,
                           *finfoP;
-       char            nulls[INDEX_MAX_KEYS];
        int                     i;
 
        heapDescriptor = RelationGetDescr(heapRelation);
@@ -152,11 +152,11 @@ CatalogIndexInsert(Relation *idescs,
                                           (AttrNumber *) index_form->indkey,
                                           heapTuple,
                                           heapDescriptor,
-                                          &datum,
+                                          datum,
                                           nulls,
                                           finfoP);
 
-               indexRes = index_insert(idescs[i], &datum, nulls,
+               indexRes = index_insert(idescs[i], datum, nulls,
                                                                &heapTuple->t_ctid, heapRelation);
                if (indexRes)
                        pfree(indexRes);
index 9898f581886743ade2a17fbe8b42cb26a758fd1e..d40f7d5055c3c312b605e69f9ed9a192440808f1 100644 (file)
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/commands/vacuum.c,v 1.80 1998/09/01 04:28:05 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/commands/vacuum.c,v 1.81 1998/09/02 23:05:25 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1156,7 +1156,7 @@ failed to add item with len = %u to page %u (free space %u, nusd %u, noff %u)",
                                        iresult = index_insert(Irel[i],
                                                                                   idatum,
                                                                                   inulls,
-                                                                                  &(newtup->t_ctid),
+                                                                                  &newtup->t_ctid,
                                                                                   onerel);
                                        if (iresult)
                                                pfree(iresult);
index f97776941cec5aa9cbc17e540ab4b0b4ff7f7bf9..e95bdce23e5e6b6032954ec5d4c75114e21a91cc 100644 (file)
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/parser/parse_target.c,v 1.27 1998/09/01 04:30:37 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/parser/parse_target.c,v 1.28 1998/09/02 23:05:27 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -467,7 +467,6 @@ MakeTargetEntryAttr(ParseState *pstate,
        int                     resdomno;
        List       *attrs = att->attrs;
        TargetEntry *tent;
-       Oid                     relid;
 
        attrname = strVal(lfirst(att->attrs));
 
index f266789c88f3e21e8f8868474a19825eea0ffc80..2e63408eaa049f09c354e815748867a55b3735b3 100644 (file)
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/storage/ipc/ipc.c,v 1.33 1998/09/01 04:31:47 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/storage/ipc/ipc.c,v 1.34 1998/09/02 23:05:30 momjian Exp $
  *
  * NOTES
  *
@@ -65,7 +65,6 @@ static struct ONEXIT
 
 static int     on_proc_exit_index,
                        on_shmem_exit_index;
-static void IpcConfigTip(void);
 
 typedef struct _PrivateMemStruct
 {
@@ -739,6 +738,7 @@ LockIsFree(int lockid)
 
 #endif  /* HAS_TEST_AND_SET */
 
+#ifdef NOT_USED
 static void
 IpcConfigTip(void)
 {
@@ -746,3 +746,4 @@ IpcConfigTip(void)
        fprintf(stderr, "shared memory or System V IPC semaphore configuration.\n");
        fprintf(stderr, "See the FAQ for more detailed information\n");
 }
+#endif
index 2496bdfdc1a0ee22187b737ecfd84893f56c3fb1..577a859077998fa57f2ebd0f9a552e21fe83152b 100644 (file)
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/tcop/pquery.c,v 1.18 1998/09/01 04:32:14 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/tcop/pquery.c,v 1.19 1998/09/02 23:05:34 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -210,7 +210,7 @@ ProcessQueryDesc(QueryDesc *queryDesc)
        Query      *parseTree;
        Plan       *plan;
        int                     operation;
-       char       *tag;
+       char       *tag = NULL;
        EState     *state;
        TupleDesc       attinfo;
 
index 291bff798cdfdcf1de29cb2e36365db5f3c2fbe8..527f041e098c392979a0e4140e3021a3e1a32200 100644 (file)
@@ -81,8 +81,6 @@ static char *opt_names[] = {
  */
 int                    pg_options[NUM_PG_OPTIONS] = {0};
 
-static int     openlog_done = 0;
-
 /*
  * Print a timestamp and a message to stdout if the trace flag
  * indexed by the flag value is set.