Cédric Villemain

adding `GRANT/REVOKE ... (COLUMN) ON TABLE TO ROLE`

5

I am looking into adding support to this privilege in citus. Before stepping in, is it something which has already been evaluated ? Specifically I wonder about possible implication with...

The main point here is to allow faster builds and regression testing. So instead of building container image with pgaf source code included we build more specialized images without source...

bad planner tree sent to executor in some MULTIEXPR scenario

1

While working on #7675 I found another bug: ``` CREATE TABLE test_ref_multiexpr ( id bigint primary key , col_int integer , col_bool bool , col_text text , col_timestamp timestamp );...

bad rewriting of query involving targetList indirection: "UPDATE SET (col2, col1, col3)"

1

A customer reported a bug from development team affecting queries with `UPDATE SET (col2, col1, col3) = (.....)` The columns are reordered following their physical ordering in pg_attribute, leading to...

WIP Fix issue #7674 about UPDATE SET(..), with indirection

26

Issue reported to Data Bene support. See #7674 The fix looks good, at least it breaks nothing and the bug not visible. I am unsure to add all regressions tests...

evaluating addition and use of PostgreSQL Injection Points in Citus

1

PostgreSQL Injection Points can be added to Citus and used for easier debugging and problem definition (test, faillure scenario, etc...) Anyone looked at that in the scope of Citus already...

WIP: Revert partially commit 4f60ddd1e and alternative fix for #1349

1

All tests are passing, and it also fixes the infinite loop outlined in #4522 WIP: I will add tests for my infinite loop

infinite loop with combination of long line and removing extra new lines fully.

3

I have reduced another case: infinite loop due to line length, "abcdef" makes the line 79 chars. I though it was related the #4515 but ... ``` static void infinite_loop(){for...

There were still few places not using the macro but directly updating the counter, assuming those were just missed.

bug and workaround fix: bad size_t assignement

7

Reduced case: ```C void recursive_bug(){if (foo()) bar(); } ``` ``` # Uncrustify-0.80.1-146-bb68ec18b eat_blanks_after_open_brace = true eat_blanks_before_close_brace = true nl_remove_extra_newlines = 2 # option(s) with 'not default' value: 118 # ```...