Hi hackers, > Anyone else have thoughts?
I came across this thread while looking for the patches that need review. My understanding of the code is limited, but I can say that I don't see anything particularly wrong with it. I can also confirm that it fixes the problem reported by the user while passing the rest of the tests. I understand the concern expressed by Robert in respect of backward compatibility. From the user's perspective, personally I would prefer a fixed bug over backward compatibility. Especially if we consider the fact that the current behaviour of cases like `select row_to_json(i) from int8_tbl i(x,y)` is not necessarily the correct one, at least it doesn't look right to me. So unless anyone has strong objections against the proposed fix or can propose a better patch, I would suggest merging it. -- Best regards, Aleksander Alekseev
