{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":4781192,"defaultBranch":"master","name":"informix_fdw","ownerLogin":"credativ","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2012-06-25T12:00:26.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/1701644?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1722434037.0","currentOid":""},"activityList":{"items":[{"before":"7d8ccd1ad8e062f091b11c5a1c4b68ff520a53b1","after":"37439823dbd6b06468e6fae4a496156e0ca430f2","ref":"refs/heads/master","pushedAt":"2024-07-31T13:51:19.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"mbanck-ntap","name":"Michael Banck","path":"/mbanck-ntap","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/157472119?s=80&v=4"},"commit":{"message":"Merge pull request #2 from psoo/DEV_0_6_2\n\nSupport PostgreSQL 17.","shortMessageHtmlLink":"Merge pull request #2 from psoo/DEV_0_6_2"}},{"before":"b518b083ccf16775e1674e7a7763fd5ebd1b270a","after":"7d8ccd1ad8e062f091b11c5a1c4b68ff520a53b1","ref":"refs/heads/master","pushedAt":"2023-08-21T13:11:01.000Z","pushType":"push","commitsCount":5,"pusher":{"login":"mbanck","name":"Michael Banck","path":"/mbanck","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1670301?s=80&v=4"},"commit":{"message":"Fix typo.","shortMessageHtmlLink":"Fix typo."}},{"before":"6742c5a72868b34fe6a3a3137454aee36dd1609d","after":"b518b083ccf16775e1674e7a7763fd5ebd1b270a","ref":"refs/heads/master","pushedAt":"2023-08-14T09:18:53.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"mbanck","name":"Michael Banck","path":"/mbanck","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1670301?s=80&v=4"},"commit":{"message":"Squash commits of devel branch DEV_0_6_0 in preparation for release REL0_6_0.\n\ncommit 0357237df15ffe16e4a73ca5835143d1973afa63\nAuthor: Bernd Helmle \nDate: Sat Aug 5 14:10:03 2023 +0000\n\n Follow up fix for issue https://github.com/credativ/informix_fdw/issues/31.\n\n Commit 285119f024a0137ba84a3fc7b825b9e2ccf107bf partially fixed our\n architecture thinko regarding query planning, but it turned out\n that we have the same issue in ifxGetForeignRelSize(). The former\n idea was to cache all informix objects created during the planning phase\n for execution later, since we rely on query statistics by using\n the query descriptors. We created them in ifxGetForeignRelSize() and\n reused these objects in ifxPlanForeignModify() later.\n\n But ifxGetForeignRelSize() isn't necessarily called, so ifxPlanForeignModify()\n needs to take care whether the objects are usable directly or still need\n to be created.\n\n I just copied the code from ifxGetForeignRelSize() without moving it into\n a dedicated function, which fixes the problem. It might be more pretty\n to have it in a dedicated function, but leave it as-is for now.\n\n Regression tests are extended to have trigger tests as described in\n issue #31.\n\ncommit db73ab94c94d77d3d56590933c7a3dd1c66d1f5a\nAuthor: Bernd Helmle \nDate: Mon Jul 24 16:37:43 2023 +0000\n\n Fix longstanding bug in issue #31.\n\n The issue describe here\n\n httpg://github.com/credativ/informix_fdw/issues/31\n\n uncovered an unfortunate mistake in our approach on how\n we plan a modify action. Formerly, ifxPlanForeignModify() was\n also responsible to create and execute the necessary query objects\n on the remote Informix server (like describing the statement and thus\n creating the required SQLDA structure).\n\n In this issue above it turned out that we are far too optimistic in\n doing this preparation work at this early stage: ifxPlanForeignModify() might not\n be called once a prepared statement settles the execution plan and thus\n we miss important things like creating and describing the statement handles.\n\n To fix this, we need to delay creating of query objects on the remote\n Informix server to ifxBeginForeignModify(), which is now responsible\n to create and describe the statement.\n\n ifxPlanForeignModify() still creates all required identifiers for\n query objects (like cursor, statement handle, ...), but this leaves\n us after the plan settled with a static reference counter (refid) used\n in the identifiers.\n I think this isn't a problem here, since afaics the refid can't be reused\n somewhere else at this point. This might be wrong, but since we pass\n regression tests at this point i leave it for now.\n\ncommit 3fdfab57056c454738c99279f1d9dbfad5e9c0e0\nAuthor: Bernd Helmle \nDate: Thu Jul 20 16:41:31 2023 +0000\n\n Update and correct README.\n\ncommit b2798e3f4866c851dc20dc4d0109d66f58dda3dc\nAuthor: Bernd Helmle \nDate: Thu Jul 20 16:07:41 2023 +0000\n\n Fix copy&paste error.\n\ncommit 67eed75976efa40f17b5452b373fb732ed088c11\nAuthor: Bernd Helmle \nDate: Thu Jul 20 15:56:46 2023 +0000\n\n Follow up commit for backward compatibility.\n\n HASH_STRINGS is available as dynahash flag beginning with PG14,\n so handle it accordingly.\n\ncommit ab2b38f5fd714acced49c9c6bac2a462a977d76d\nAuthor: Bernd Helmle \nDate: Thu Jul 20 10:21:08 2023 +0000\n\n Fix compile issues caused by changes made for PG16 support.\n\ncommit 1f71169f36d121260ec4b27c106d54c04ca4e24a\nAuthor: Bernd Helmle \nDate: Thu Jul 20 09:59:12 2023 +0000\n\n Support PostgreSQL 16.\n\n Including commit 76a08e2e735584131e383c24e5bcdf3ff34da5e8 this introduces\n support for the upcoming new major release 16 of PostgreSQL.\n\n Commit a61b1f74823c9c4f79c95226a461f1e7a367764b reworked the relation\n permission checking infrastructure in PostgreSQL which forces us to cope\n with them properly.\n\n No new features yet and needs testing against all supported older\n major releases (as of this commit these are 10, 11, 12, 13, 14, 15).\n\ncommit fc137b2c258ecc21cc1290da33417b04780abf5f\nAuthor: Bernd Helmle \nDate: Thu Jul 20 09:56:01 2023 +0000\n\n Make HASH_STRINGS in connection cache explicit.\n\n Starting with PostgreSQL 16 dynahash's implementation forces\n explicit usage of HASH_STRINGS when creating the hash table we use\n for storing Informix connection handles. Since the lookup key is the\n connection identifier composed by strings up to IFX_CONNAME_LEN, we\n must set HASH_STRINGS accordingly, otherwise we run into an\n assertion failure.","shortMessageHtmlLink":"Squash commits of devel branch DEV_0_6_0 in preparation for release R…"}},{"before":"c41179fb71397582b5a7b70f56a88f14d4d4efbc","after":"6742c5a72868b34fe6a3a3137454aee36dd1609d","ref":"refs/heads/master","pushedAt":"2023-08-11T12:20:42.000Z","pushType":"pr_merge","commitsCount":4,"pusher":{"login":"psoo","name":"Bernd Helmle","path":"/psoo","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/362650?s=80&v=4"},"commit":{"message":"Merge pull request #38 from psoo/master\n\nMerge upstream development","shortMessageHtmlLink":"Merge pull request #38 from psoo/master"}}],"hasNextPage":false,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEjh9vQQA","startCursor":null,"endCursor":null}},"title":"Activity · credativ/informix_fdw"}