morningman opened a new pull request, #28163:
URL: https://github.com/apache/doris/pull/28163
## Proposed changes
This PR #27515 change the logic if Table's `isPartitioned()` method.
But this method has 2 usages:
1. To check whether a table is range or list partitioned, for some DML
operation such as Alter, Export.
For this case, it should return true if the table is range or list
partitioned. even if it has only
one partition and one buckets.
2. To check whether the data is distributed (either by partitions or by
buckets), for query planner.
For this case, it should return true if table has more than one bucket.
Even if this table is not
range or list partitioned, if it has more than one bucket, it should
return true.
So we should separate this method into 2, for different usages.
Otherwise, it may cause some unreasonable plan shape
## Further comments
If this is a relatively large or complex change, kick off the discussion at
[d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you
chose the solution you did and what alternatives you considered, etc...
--
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: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
