You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by ch...@apache.org on 2019/03/01 00:16:15 UTC
[phoenix] branch master updated: PHOENIX-374: Enable access to
dynamic columns in * or cf.* selection (Addendum)
This is an automated email from the ASF dual-hosted git repository.
chinmayskulkarni pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/phoenix.git
The following commit(s) were added to refs/heads/master by this push:
new af9ae19 PHOENIX-374: Enable access to dynamic columns in * or cf.* selection (Addendum)
af9ae19 is described below
commit af9ae1933b48b00e7ff9c4d8417678338ce4a18b
Author: Chinmay Kulkarni <ch...@gmail.com>
AuthorDate: Thu Feb 28 15:47:42 2019 -0800
PHOENIX-374: Enable access to dynamic columns in * or cf.* selection (Addendum)
---
phoenix-core/src/main/java/org/apache/phoenix/schema/PTableImpl.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/schema/PTableImpl.java b/phoenix-core/src/main/java/org/apache/phoenix/schema/PTableImpl.java
index a7936e0..cd961da 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/schema/PTableImpl.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/schema/PTableImpl.java
@@ -1291,7 +1291,9 @@ public class PTableImpl implements PTable {
}
String fam = Bytes.toString(family);
if (column.isDynamic()) {
- this.colFamToDynamicColumnsMapping.putIfAbsent(fam, new ArrayList<>());
+ if (!this.colFamToDynamicColumnsMapping.containsKey(fam)) {
+ this.colFamToDynamicColumnsMapping.put(fam, new ArrayList<>());
+ }
this.colFamToDynamicColumnsMapping.get(fam).add(column);
}
}