You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@htorun Every thing works fantastic besides one thing. When there is a column of number datatype in SQL table, it creates some parsing problem. Please refer this image:
I have tried to make necessary changes and now its working fine. Please review and let me know what you think!
here is the screenshot of what result i got after changes i made
Actually, I just realized that you included the NUMERIC data type under CATEGORY_INTEGER. Since NUMERIC is like DECIMAL and can store a decimal number, in order to be formatted correctly it should go under CATEGORY_DOUBLE. (I also realized that CATEGORY_DECIMAL or CATEGORY_REAL would be a better name for CATEGORY_DOUBLE :) )
I also encountered this problem on Oracle, and the fix I had was to also get the scale of the column from the ResultSetMetadata, store that on the Column, and change whichCategory to operate on a Column instead of just a int type since choosing the correct java.sql.Type requires more information (e.g. Type.NUMERIC can either be an Integer or Decimal depending on the scale)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@htorun Every thing works fantastic besides one thing. When there is a column of number datatype in SQL table, it creates some parsing problem. Please refer this image:

I have tried to make necessary changes and now its working fine. Please review and let me know what you think!

here is the screenshot of what result i got after changes i made