Jim Apple has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9192 )

Change subject: IMPALA-6204: Remove external DataSource
......................................................................


Patch Set 5:

> > Was there ever an announcement that this was deprecated?
 >
 > Is https://issues.apache.org/jira/browse/IMPALA-6204 sufficient?

TBH, I'm not thrilled about that. For instance, there is no justification on 
that ticket about why, and no discussion of pros and cons.

While this is in review, Maybe Philip could start a discussion on user@ 
explaining why and asking for feedback. Maybe it will come back unanimous after 
a short amount of time, but I think it's worth a discussion, given how badly 
this could break workloads of existing users. (Obviously they could ETL into 
Parquet or stay on the 2.x branch.)


--
To view, visit http://gerrit.cloudera.org:8080/9192
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I02a3a6740466ed7372b71d948c705b30886dcfb6
Gerrit-Change-Number: 9192
Gerrit-PatchSet: 5
Gerrit-Owner: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com>
Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogian...@cloudera.com>
Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Zach Amsden <zams...@cloudera.com>
Gerrit-Comment-Date: Tue, 06 Feb 2018 19:19:52 +0000
Gerrit-HasComments: No

Reply via email to