jayzhan211 commented on PR #25576:
URL: https://github.com/apache/datafusion/pull/25576#issuecomment-6039244336

   Merge the main since:
   
   Tag 40 is taken on `main` since #24227 (`bool enable_rle_to_dictionary = 
40;`), which merged after this branch's last push. The merge is textually 
clean, so GitHub shows it as mergeable and CI is green, but the merged tree 
doesn't compile:
   
   ```sh
   $ cargo check -p datafusion-proto-common   # this branch merged with current 
main
   error: proc-macro derive panicked
     = help: message: called `Result::unwrap()` on an `Err` value: message 
ParquetOptions has multiple fields with tag 40
   ```
   
   Fix: rebase on `main`, take the next free tag, and regenerate both generated 
copies:
   ```diff
   -  bool write_row_group_number_distinct_values = 40; // default = false
   +  bool write_row_group_number_distinct_values = 41; // default = false
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to