virajjasani commented on PR #2209: URL: https://github.com/apache/phoenix/pull/2209#issuecomment-3041055668
> Have you looked into a way to reuse the existing logic for building change images instead of building a parallel logic in the new CDCCompactionUtil class? Yes i did but there are 2 concerns: 1. CDCTableInfo cannot be directly constructed from data table PTable. Currently, it only provides a way to get initialized using serialization and deserialization path. This makes it difficult to directly use it at the server side. 2. Even if CDCTableInfo and CDCColumnInfo could be used directly at the server side, they provide a redundant layer on top of what can already be constructed with minimal/simpler efforts i.e. mutation for uncovered index with pre-built image. -- 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: issues-unsubscr...@phoenix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
