David,

Thank you for the bug report.  You should see notification when it is addressed.

Rob

On Fri, Oct 12, 2012 at 8:09 PM, David Pinho <david.pinho@gmail.com> wrote:
So anyway i pulled the source and the following seems to fix it:

### Eclipse Workspace Patch 1.0
#P oracle
Index: src/main/java/net/sourceforge/squirrel_sql/plugins/oracle/tokenizer/OracleQueryTokenizer.java
===================================================================
--- src/main/java/net/sourceforge/squirrel_sql/plugins/oracle/tokenizer/OracleQueryTokenizer.java       (revision
6682)
+++ src/main/java/net/sourceforge/squirrel_sql/plugins/oracle/tokenizer/OracleQueryTokenizer.java       (working
copy)
@@ -239,7 +239,7 @@
                for (Iterator<String> iter = _queries.iterator(); iter.hasNext();)
                {
                        String next = iter.next();
-                       if (next.startsWith(sep))
+                       if (next.startsWith(sep) && !next.startsWith("/*"))
                        {
                                tmp.add(sep);
                                String[] parts = next.split(sep + "\\n+");

Tell me what you think...


On Fri, Oct 12, 2012 at 4:23 PM, David Pinho <david.pinho@gmail.com> wrote:
> Done.
>
> Added the stacktrace from SQuirreL logs:
>
> 2012-10-12 16:17:28,429 [Thread-5] ERROR
> net.sourceforge.squirrel_sql.client.session.MessagePanel  -
> privateShowMessage: Exception was Warning: execution completed with
> warning
> java.sql.SQLWarning: Warning: execution completed with warning
>         at oracle.jdbc.driver.DatabaseError.addSqlWarning(DatabaseError.java:945)
>         at oracle.jdbc.driver.DatabaseError.addSqlWarning(DatabaseError.java:993)
>         at oracle.jdbc.driver.DatabaseError.addSqlWarning(DatabaseError.java:1008)
>         at oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:220)
>         at oracle.jdbc.driver.T4CStatement.executeForRows(T4CStatement.java:1033)
>         at oracle.jdbc.driver.OracleStatement.doExecuteWithTimeout(OracleStatement.java:1329)
>         at oracle.jdbc.driver.OracleStatement.executeInternal(OracleStatement.java:1909)
>         at oracle.jdbc.driver.OracleStatement.execute(OracleStatement.java:1871)
>         at oracle.jdbc.driver.OracleStatementWrapper.execute(OracleStatementWrapper.java:318)
>         at net.sourceforge.squirrel_sql.client.session.SQLExecuterTask.processQuery(SQLExecuterTask.java:410)
>         at net.sourceforge.squirrel_sql.client.session.SQLExecuterTask.run(SQLExecuterTask.java:220)
>         at net.sourceforge.squirrel_sql.fw.util.TaskExecuter.run(TaskExecuter.java:82)
>         at java.lang.Thread.run(Unknown Source)
>
>
> On Fri, Oct 12, 2012 at 12:11 PM, Robert Manning
> <robert.m.manning@gmail.com> wrote:
>> That would be a bug in the Oracle plugin's query tokenizer.   Can you put in
>> a bug ticket for that here:
>>
>> http://sourceforge.net/tracker/?group_id=28383&atid=393414
>>
>> Rob
>>
>> On Fri, Oct 12, 2012 at 4:42 AM, David Pinho <david.pinho@gmail.com> wrote:
>>>
>>> Hi all,
>>>
>>> I'm seeing a strange behavior (well strange for me anyway). When
>>> creating a package in an Oracle DB:
>>>
>>> If i execute this in SQuirreL:
>>>
>>> CREATE OR REPLACE PACKAGE foo AS
>>>
>>> /* comment */
>>> c1 CONSTANT INTEGER := 1;
>>>
>>> FUNCTION f1 (arg1 IN NUMBER, arg2 IN DATE) RETURN INTEGER;
>>>
>>> END foo;
>>> /
>>>
>>> It will execute fine and create the package. But if i change the
>>> position of the comment line:
>>>
>>> CREATE OR REPLACE PACKAGE foo AS
>>>
>>> c1 CONSTANT INTEGER := 1;
>>> /* comment */
>>>
>>> FUNCTION f1 (arg1 IN NUMBER, arg2 IN DATE) RETURN INTEGER;
>>>
>>> END foo;
>>> /
>>>
>>> it will fail with the error:
>>> Query 1 of 4, Rows read: 0, Elapsed time (seconds) - Total: 0.02, SQL
>>> query: 0.02, Reading results: 0
>>> Error: Invalid SQL type: sqlKind = 0
>>> SQLState:  99999
>>> ErrorCode: 17439
>>> Position: 0
>>> Error occured in:
>>> /* comment *
>>>
>>>
>>> If i run it in SQLPlus it will run just fine. Any ideias what this
>>> might be or how i can get around it?
>>>
>>> I'm using Oracle 10.2.0.4.0 and SQuirreL 3.4.0 (Oracle Thin Driver).
>>>
>>>
>>> Tks,
>>>
>>> David
>>>
>>>
>>>
>>>
>>> --
>>> http://davidpinho.net
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> Don't let slow site performance ruin your business. Deploy New Relic APM
>>> Deploy New Relic app performance management and know exactly
>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>> http://p.sf.net/sfu/newrelic-dev2dev
>>> _______________________________________________
>>> Squirrel-sql-users mailing list
>>> Squirrel-sql-users@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/squirrel-sql-users
>>
>>
>>
>> ------------------------------------------------------------------------------
>> Don't let slow site performance ruin your business. Deploy New Relic APM
>> Deploy New Relic app performance management and know exactly
>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>> http://p.sf.net/sfu/newrelic-dev2dev
>> _______________________________________________
>> Squirrel-sql-users mailing list
>> Squirrel-sql-users@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/squirrel-sql-users
>>
>
>
>
> --
> http://davidpinho.net



--
http://davidpinho.net

------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Squirrel-sql-users mailing list
Squirrel-sql-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/squirrel-sql-users