[GitHub] [incubator-doris] morningman commented on pull request #3604: (#3464) fix Query failed when fact table has no data in join case

2020-05-29 Thread GitBox
morningman commented on pull request #3604: URL: https://github.com/apache/incubator-doris/pull/3604#issuecomment-635962189 > I think we can set `partition join` as default when we're unable to estimate the cost accurately and broadcast join cost equals partition join cost and without

[GitHub] [incubator-doris] morningman commented on pull request #3604: (#3464) fix Query failed when fact table has no data in join case

2020-05-27 Thread GitBox
morningman commented on pull request #3604: URL: https://github.com/apache/incubator-doris/pull/3604#issuecomment-635091296 > @morningman > The key point is that it's currently impossible to accurately estimate the size of the memory footprint at runtime according to table

[GitHub] [incubator-doris] morningman commented on pull request #3604: (#3464) fix Query failed when fact table has no data in join case

2020-05-22 Thread GitBox
morningman commented on pull request #3604: URL: https://github.com/apache/incubator-doris/pull/3604#issuecomment-632976533 > 1 make HASH_TBL_SPACE_OVERHEAD a configurable param,but it seems not a common solution, because I'm not sure what factors can affect the real hash table size. >