kaixuanzhang123 opened a new issue, #5870:
URL: https://github.com/apache/paimon/issues/5870

   ### Search before asking
   
   - [x] I searched in the [issues](https://github.com/apache/paimon/issues) 
and found nothing similar.
   
   
   ### Paimon version
   
   paimon master branch, commit id: 9ec10007bc33e9849dcdaa42b724d5d05bd7bcc3
   
   ### Compute Engine
   
   flink 1.17
   
   ### Minimal reproduce step
   
   1. set branch table external data-file.external-paths: "/tmp/a"
   2. write data
   3. set external data-file.external-paths: "/tmp/b"
   4. write data
   5. delete table
   
   ### What doesn't meet your expectations?
   
   `public void dropTable(Identifier identifier, boolean ignoreIfNotExists)
               throws TableNotExistException {
           checkNotBranch(identifier, "dropTable");
           checkNotSystemTable(identifier, "dropTable");
   
           List<Path> externalPaths = new ArrayList<>();
           try {
               Table table = getTable(identifier);
               if (table instanceof FileStoreTable) {
                   FileStoreTable fileStoreTable = (FileStoreTable) table;
                   externalPaths =
                           fileStoreTable.schemaManager().listAll().stream()
                                   .map(
                                           schema ->
                                                   schema.toSchema()
                                                           .options()
                                                           
.get(DATA_FILE_EXTERNAL_PATHS.key()))
                                   .filter(Objects::nonNull)
                                   .flatMap(externalPath -> 
Arrays.stream(externalPath.split(",")))
                                   .map(Path::new)
                                   .distinct()
                                   .collect(Collectors.toList());
               }
           } catch (TableNotExistException e) {
               if (ignoreIfNotExists) {
                   return;
               }
               throw new TableNotExistException(identifier);
           }
   
           dropTableImpl(identifier, externalPaths);
       }`
   branch table external path may be not delete
   
   ### Anything else?
   
   delete branch  file
   
   ### Are you willing to submit a PR?
   
   - [x] I'm willing to submit a PR!


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

Reply via email to