[
https://issues.apache.org/jira/browse/PHOENIX-2852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15261304#comment-15261304
]
Hudson commented on PHOENIX-2852:
---------------------------------
FAILURE: Integrated in Phoenix-master #1204 (See
[https://builds.apache.org/job/Phoenix-master/1204/])
PHOENIX-2852 Dropping columns could trigger upgrade code on server side
(tdsilva: rev 1fb6b1b795609f7f2df35c2482cd4ec95bcc0cbe)
*
phoenix-core/src/main/java/org/apache/hadoop/hbase/regionserver/LocalIndexSplitter.java
*
phoenix-core/src/main/java/org/apache/hadoop/hbase/regionserver/IndexHalfStoreFileReaderGenerator.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/FormatToKeyValueReducer.java
*
phoenix-core/src/main/java/org/apache/phoenix/index/PhoenixIndexFailurePolicy.java
*
phoenix-core/src/main/java/org/apache/phoenix/coprocessor/MetaDataRegionObserver.java
* phoenix-core/src/main/java/org/apache/phoenix/util/QueryUtil.java
* phoenix-core/src/main/java/org/apache/phoenix/trace/PhoenixMetricsSink.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/FormatToBytesWritableMapper.java
*
phoenix-core/src/main/java/org/apache/phoenix/coprocessor/MetaDataEndpointImpl.java
> Dropping columns could trigger upgrade code on server side
> ----------------------------------------------------------
>
> Key: PHOENIX-2852
> URL: https://issues.apache.org/jira/browse/PHOENIX-2852
> Project: Phoenix
> Issue Type: Bug
> Reporter: Samarth Jain
> Assignee: Thomas D'Silva
> Attachments: PHOENIX-2852.patch
>
>
> In MetadataEndPointImpl.java, I noticed that we are obtaining a Phoenix
> connection by going through the Phoenix driver. This could be dangerous as it
> could end up triggering the upgrade code.
> {code}
> // check if the view where expression contains the column being dropped and
> prevent
> // it
> if (existingViewColumn != null && view.getViewStatement() !=
> null) {
> ParseNode viewWhere =
> new
> SQLParser(view.getViewStatement()).parseQuery().getWhere();
> PhoenixConnection conn=null;
> try {
> conn =
> QueryUtil.getConnection(env.getConfiguration()).unwrap(
> PhoenixConnection.class);
> } catch (ClassNotFoundException e) {
> }
> {code}
> [~jamestaylor] has added a nice way of getting a phoenix connection without
> running the upgrade by doing something like this:
> {code}
> final Properties props = new Properties();
> props.setProperty(PhoenixRuntime.NO_UPGRADE_ATTRIB,
> Boolean.TRUE.toString());
> conn = DriverManager.getConnection(getJdbcUrl(env),
> props).unwrap(PhoenixConnection.class);
> {code}
> [~tdsilva] - would you mind taking a look? Thanks!
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
