Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitaf7d181

Browse files
committed
Fix pgstatindex() to give consistent results for empty indexes.
For an empty index, the pgstatindex() function would compute 0.0/0.0 forits avg_leaf_density and leaf_fragmentation outputs. On machines thatfollow the IEEE float arithmetic standard with any care, that results ina NaN. However, per report from Rushabh Lathia, Microsoft couldn'tmanage to get this right, so you'd get a bizarre error on Windows.Fix by forcing the results to be NaN explicitly, rather than relying onthe division operator to give that or the snprintf function to print itcorrectly. I have some doubts that this is really the most usefuldefinition, but it seems better to remain backward-compatible withthose platforms for which the behavior wasn't completely broken.Back-patch to 8.2, since the code is like that in all current releases.
1 parentf18795e commitaf7d181

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

‎contrib/pgstattuple/pgstatindex.c

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -235,9 +235,17 @@ pgstatindex(PG_FUNCTION_ARGS)
235235
values[j]=palloc(32);
236236
snprintf(values[j++],32,INT64_FORMAT,indexStat.deleted_pages);
237237
values[j]=palloc(32);
238-
snprintf(values[j++],32,"%.2f",100.0- (double)indexStat.free_space / (double)indexStat.max_avail*100.0);
238+
if (indexStat.max_avail>0)
239+
snprintf(values[j++],32,"%.2f",
240+
100.0- (double)indexStat.free_space / (double)indexStat.max_avail*100.0);
241+
else
242+
snprintf(values[j++],32,"NaN");
239243
values[j]=palloc(32);
240-
snprintf(values[j++],32,"%.2f", (double)indexStat.fragments / (double)indexStat.leaf_pages*100.0);
244+
if (indexStat.leaf_pages>0)
245+
snprintf(values[j++],32,"%.2f",
246+
(double)indexStat.fragments / (double)indexStat.leaf_pages*100.0);
247+
else
248+
snprintf(values[j++],32,"NaN");
241249

242250
tuple=BuildTupleFromCStrings(TupleDescGetAttInMetadata(tupleDesc),
243251
values);

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp