gianm commented on issue #7728: fix Select query fails with columns called "timestamp" bug URL: https://github.com/apache/incubator-druid/pull/7728#issuecomment-495016935 How about instead fixing this by keeping time in `timestamp`, but then also dropping any dimension/metric that is named `timestamp` (instead of overwriting the time)? That would mean there's no incompatibility. I don't think the API churn is a good thing in this case. It will make migration tough. If we're going to force people to change their API calls, they might as well switch to Scan, which is better than Select.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
