Re: [PR] [SPARK-44685][SQL] Remove deprecated Catalog#createExternalTable [spark]

2023-11-29 Thread via GitHub


LuciferYang commented on PR #42356:
URL: https://github.com/apache/spark/pull/42356#issuecomment-1831874765

   > > > > We should only remove an API if there is clear evidence showing that 
no one is using it.
   > > > 
   > > > 
   > > > Thanks, I will open a discussion in the mailing list after 3.5.0 
released.
   > > 
   > > 
   > > @Hisoka-X Will you still initiate this discussion?
   > 
   > I think @srowen already open a discussion in 
https://lists.apache.org/thread/yf2wvypvkqk7rqtv4s641ygbk4gr3tc2
   
   OK, I see, thanks


-- 
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: reviews-unsubscr...@spark.apache.org

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


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



Re: [PR] [SPARK-44685][SQL] Remove deprecated Catalog#createExternalTable [spark]

2023-11-29 Thread via GitHub


Hisoka-X commented on PR #42356:
URL: https://github.com/apache/spark/pull/42356#issuecomment-1831790628

   > > > We should only remove an API if there is clear evidence showing that 
no one is using it.
   > > 
   > > 
   > > Thanks, I will open a discussion in the mailing list after 3.5.0 
released.
   > 
   > @Hisoka-X Will you still initiate this discussion?
   
   I think @srowen already open a discussion in 
https://lists.apache.org/thread/yf2wvypvkqk7rqtv4s641ygbk4gr3tc2


-- 
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: reviews-unsubscr...@spark.apache.org

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


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



Re: [PR] [SPARK-44685][SQL] Remove deprecated Catalog#createExternalTable [spark]

2023-11-29 Thread via GitHub


LuciferYang commented on PR #42356:
URL: https://github.com/apache/spark/pull/42356#issuecomment-1831418085

   > > We should only remove an API if there is clear evidence showing that no 
one is using it.
   > 
   > Thanks, I will open a discussion in the mailing list after 3.5.0 released.
   
   @Hisoka-X Will you still initiate this discussion?
   
   


-- 
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: reviews-unsubscr...@spark.apache.org

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


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



Re: [PR] [SPARK-44685][SQL] Remove deprecated Catalog#createExternalTable [spark]

2023-11-16 Thread via GitHub


github-actions[bot] closed pull request #42356: [SPARK-44685][SQL] Remove 
deprecated Catalog#createExternalTable
URL: https://github.com/apache/spark/pull/42356


-- 
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: reviews-unsubscr...@spark.apache.org

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


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



Re: [PR] [SPARK-44685][SQL] Remove deprecated Catalog#createExternalTable [spark]

2023-11-15 Thread via GitHub


github-actions[bot] commented on PR #42356:
URL: https://github.com/apache/spark/pull/42356#issuecomment-1813504669

   We're closing this PR because it hasn't been updated in a while. This isn't 
a judgement on the merit of the PR in any way. It's just a way of keeping the 
PR queue manageable.
   If you'd like to revive this PR, please reopen it and ask a committer to 
remove the Stale tag!


-- 
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: reviews-unsubscr...@spark.apache.org

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


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org