[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vineet Garg updated HIVE-20847: --- Resolution: Fixed Fix Version/s: 4.0.0 Status: Resolved (was: Patch Available) Pushed to master. Thanks [~belugabehr] > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Fix For: 4.0.0 > > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch, HIVE-20847.3.patch, HIVE-20847.4.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Patch Available (was: Open) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch, HIVE-20847.3.patch, HIVE-20847.4.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Open (was: Patch Available) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch, HIVE-20847.3.patch, HIVE-20847.4.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Attachment: HIVE-20847.4.patch > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch, HIVE-20847.3.patch, HIVE-20847.4.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Open (was: Patch Available) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch, HIVE-20847.3.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Patch Available (was: Open) Attaching same patch again. Hopefully unit tests pass. > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch, HIVE-20847.3.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Attachment: HIVE-20847.3.patch > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch, HIVE-20847.3.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Attachment: (was: HIVE-20847.3.patch) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch, HIVE-20847.3.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Attachment: HIVE-20847.3.patch > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch, HIVE-20847.3.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Affects Version/s: (was: 3.1.0) 3.2.0 > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Patch Available (was: Open) Added a new patch to fix more checkstyle errors. Also, changed data structures to used Array-Based instead of Linked structures where possible. > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Open (was: Patch Available) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Attachment: HIVE-20847.2.patch > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 4.0.0, 3.2.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch, HIVE-20847.2.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Attachment: (was: HIVE-20847.1.patch) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Patch Available (was: Open) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Open (was: Patch Available) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Attachment: HIVE-20847.1.patch > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Open (was: Patch Available) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Attachment: HIVE-20847.1.patch > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Patch Available (was: Open) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch, > HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Patch Available (was: Open) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Status: Open (was: Patch Available) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Attachment: HIVE-20847.1.patch > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HIVE-20847) Review of NullScan Code
[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] BELUGA BEHR updated HIVE-20847: --- Summary: Review of NullScan Code (was: Review of NullScanCode) > Review of NullScan Code > --- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer >Affects Versions: 3.1.0, 4.0.0 >Reporter: BELUGA BEHR >Assignee: BELUGA BEHR >Priority: Minor > Attachments: HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList > aliasesAffected, ArrayList aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList allowed = new ArrayList(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)