[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ramkrishna.s.vasudevan updated HBASE-16729: --- Resolution: Fixed Status: Resolved (was: Patch Available) Pushed v3 to master. Thanks for the patch [~chia7712]. Thanks for the comment Stack. Marked as incompatible change. > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch, > HBASE-16729.v2.patch, HBASE-16729.v3.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Attachment: HBASE-16729.v3.patch address [~ram_krish]'s comments > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch, > HBASE-16729.v2.patch, HBASE-16729.v3.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Status: Patch Available (was: Open) > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch, > HBASE-16729.v2.patch, HBASE-16729.v3.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Status: Open (was: Patch Available) > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch, > HBASE-16729.v2.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ramkrishna.s.vasudevan updated HBASE-16729: --- Hadoop Flags: Incompatible change Release Note: Empty filter list will behave as when there is no filter added. This change is a behavioral change for those who rely on Empty filter list. (was: Empty FilterList will do nothing) > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch, > HBASE-16729.v2.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Attachment: HBASE-16729.v2.patch > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch, > HBASE-16729.v2.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Status: Patch Available (was: Open) > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch, > HBASE-16729.v2.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Status: Open (was: Patch Available) > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Attachment: HBASE-16729.v1.patch FilterList shouldn't be a subclass of FilterBase because FilterList doesn't support the FilterBase#createFilterFromArguments. > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Status: Patch Available (was: Open) > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch, HBASE-16729.v1.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Affects Version/s: 2.0.0 > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Fix Version/s: 2.0.0 > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Affects Versions: 2.0.0 >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Fix For: 2.0.0 > > Attachments: HBASE-16729.v0.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Status: Open (was: Patch Available) > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Reporter: ChiaPing Tsai >Assignee: ChiaPing Tsai >Priority: Trivial > Attachments: HBASE-16729.v0.patch > > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (HBASE-16729) Define the behavior of (default) empty FilterList
[ https://issues.apache.org/jira/browse/HBASE-16729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ChiaPing Tsai updated HBASE-16729: -- Description: Current empty FilterList filters all data, because the FilterList#isFamilyEssential always returns false which causes the null cell retrieved by RegionScannerImpl.storeHeap. It seems to me that empty FilterList should do nothing, because the following code is common. {noformat} private static Filter makeFilter() { FilterList filterList = new FilterList (); for (some conditions) { // add some filters. Or nothing to add. } return filterList; } {noformat} If we keep the current logic which filters all data, we should add enough comments to explain it. Or add the FilterList#size() or FilterList#empty() for preventing filtering all data. Any comments? Thanks. was: Current empty FilterList filters all data, because the FilterList#isFamilyEssential always returns false which causes the null cell retrieved by RegionScannerImpl.storeHeap. It seems to me that empty FilterList should do nothing, because the following code is common. {noformat} private static Filter makeFilter() { FilterList filterList = new FilterList (); for (some conditions) { // add some filters. Or nothing to add. } return filterList; } {noformat} If we keep the current logic which filters all data, we should add enough comments to explain it. Or add the FilterList#size() or FilterList#empty() for preventing filtering all data. > Define the behavior of (default) empty FilterList > - > > Key: HBASE-16729 > URL: https://issues.apache.org/jira/browse/HBASE-16729 > Project: HBase > Issue Type: Wish >Reporter: ChiaPing Tsai >Priority: Trivial > > Current empty FilterList filters all data, because the > FilterList#isFamilyEssential always returns false which causes the null cell > retrieved by RegionScannerImpl.storeHeap. > It seems to me that empty FilterList should do nothing, because the following > code is common. > {noformat} > private static Filter makeFilter() { > FilterList filterList = new FilterList (); > for (some conditions) { > // add some filters. Or nothing to add. > } > return filterList; > } > {noformat} > If we keep the current logic which filters all data, we should add enough > comments to explain it. Or add the FilterList#size() or FilterList#empty() > for preventing filtering all data. > Any comments? Thanks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)