You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@texera.apache.org by "aglinxinyuan (via GitHub)" <gi...@apache.org> on 2025/09/21 02:18:00 UTC

[PR] chore: Remove duplicate JOOQ generator and libraries [texera]

aglinxinyuan opened a new pull request, #3748:
URL: https://github.com/apache/texera/pull/3748

   Since the JOOQ code generator logic was merged into the sbt compiler in PR #3746, we no longer need a separate JOOQ generator class. This PR removes the redundant JOOQ generator along with its related libraries.
   
   We chose not to remove everything in the same PR to ensure the new generator worked as expected before deprecating the old one, and to keep each PR minimal and focused.


-- 
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: dev-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] chore: Remove duplicate JOOQ generator and libraries [texera]

Posted by "aglinxinyuan (via GitHub)" <gi...@apache.org>.
aglinxinyuan commented on PR #3748:
URL: https://github.com/apache/texera/pull/3748#issuecomment-3551175258

   This PR is ready. @bobbai00, please review it. 


-- 
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: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] chore: Remove duplicate JOOQ generator and libraries [texera]

Posted by "seongjinyoon (via GitHub)" <gi...@apache.org>.
seongjinyoon commented on PR #3748:
URL: https://github.com/apache/texera/pull/3748#issuecomment-3536643703

   > Just checking in on this — @seongjinyoon, any updates?
   
   I haven't started on this yet. I can prioritize it and give an update by Monday.


-- 
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: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] chore: Remove duplicate JOOQ generator and libraries [texera]

Posted by "aglinxinyuan (via GitHub)" <gi...@apache.org>.
aglinxinyuan commented on PR #3748:
URL: https://github.com/apache/texera/pull/3748#issuecomment-3413561551

   > shall we merge this PR now?
   
   Waiting for @seongjinyoon to add an sbt shortcut to run JOOQ generator in case developers want to run it manually. It's not urgent.


-- 
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: dev-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] chore: Remove duplicate JOOQ generator and libraries [texera]

Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on PR #3748:
URL: https://github.com/apache/texera/pull/3748#issuecomment-3536126647

   Just checking in on this — @seongjinyoon, any updates?


-- 
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: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] chore: Remove duplicate JOOQ generator and libraries [texera]

Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on PR #3748:
URL: https://github.com/apache/texera/pull/3748#issuecomment-3413555095

   shall we merge this PR now? 


-- 
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: dev-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] chore: Remove duplicate JOOQ generator and libraries [texera]

Posted by "chenlica (via GitHub)" <gi...@apache.org>.
chenlica commented on PR #3748:
URL: https://github.com/apache/texera/pull/3748#issuecomment-3316080917

   @bobbai00 Please review this PR first.


-- 
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: dev-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] chore: Remove duplicate JOOQ generator and libraries [texera]

Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on PR #3748:
URL: https://github.com/apache/texera/pull/3748#issuecomment-3593011430

   @bobbai00 can you please review this?


-- 
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: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org