Re: [I] include Postgres driver in iceberg-rest-fixture image [iceberg]

2025-12-22 Thread via GitHub


raphaelauv commented on issue #14802:
URL: https://github.com/apache/iceberg/issues/14802#issuecomment-3682031662

   I don't understand 
   
   Could you explain exactly what other solution you see than adding the driver 
to the dockerfile
   
   Thank you


-- 
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]



Re: [I] include Postgres driver in iceberg-rest-fixture image [iceberg]

2025-12-22 Thread via GitHub


manuzhang commented on issue #14802:
URL: https://github.com/apache/iceberg/issues/14802#issuecomment-3681997363

   I mean passing it as an argument to 
https://github.com/apache/iceberg/blob/main/docker/iceberg-rest-fixture/Dockerfile


-- 
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]



Re: [I] include Postgres driver in iceberg-rest-fixture image [iceberg]

2025-12-08 Thread via GitHub


raphaelauv commented on issue #14802:
URL: https://github.com/apache/iceberg/issues/14802#issuecomment-3630289379

   I don't understand , what do you mean by configurable ?
   
   currently I do
   
   ```dockerfile
   FROM apache/iceberg-rest-fixture:1.10.0
   USER root
   RUN curl -L -o /usr/lib/iceberg-rest/postgresql.jar \
   https://jdbc.postgresql.org/download/postgresql-42.7.3.jar \
&& chown iceberg:iceberg /usr/lib/iceberg-rest/postgresql.jar
   USER iceberg:iceberg
   CMD ["java", "-cp", "iceberg-rest-adapter.jar:postgresql.jar", 
"org.apache.iceberg.rest.RESTCatalogServer"]
   ```


-- 
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]



Re: [I] include Postgres driver in iceberg-rest-fixture image [iceberg]

2025-12-08 Thread via GitHub


manuzhang commented on issue #14802:
URL: https://github.com/apache/iceberg/issues/14802#issuecomment-3630059717

   Can we make it configurable?


-- 
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]



[I] include Postgres driver in iceberg-rest-fixture image [iceberg]

2025-12-08 Thread via GitHub


raphaelauv opened a new issue, #14802:
URL: https://github.com/apache/iceberg/issues/14802

   ### Feature Request / Improvement
   
   by default the image use an sqlite db 
   
   would be great that the image can directly use a remote postgres without 
need to create a custom image
   
   because currently doing
   
   CATALOG_URI=jdbc:postgresql://iceberg_rest-db:5432/iceberg_restdb
   
   fail with 2025-12-08T23:34:43.814046921Z Caused by: java.sql.SQLException: 
No suitable driver found for 
jdbc:postgresql://iceberg_rest-db:5432/iceberg_restdb
   
   
   ### Query engine
   
   None
   
   ### Willingness to contribute
   
   - [x] I can contribute this improvement/feature independently
   - [ ] I would be willing to contribute this improvement/feature with 
guidance from the Iceberg community
   - [ ] I cannot contribute this improvement/feature at this time


-- 
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]